Skip to content

Fix/249: modify yml and fix loading backend url from .env#5

Merged
Vektor19 merged 1 commit into
mainfrom
feat/249/prepare-client-to-deploy
Dec 28, 2025
Merged

Fix/249: modify yml and fix loading backend url from .env#5
Vektor19 merged 1 commit into
mainfrom
feat/249/prepare-client-to-deploy

Conversation

@Vektor19
Copy link
Copy Markdown
Contributor

dev
##GitHub

#249 project-studying-dotnet/Streetcode-Server-November-2025#249

Summary of change

This pull request focuses on improving deployment configuration and backend API connectivity. The main changes include updating the Azure Static Web Apps workflow to use a more generic secret and simplifying the backend API URL configuration in the frontend code.

Deployment workflow updates:

  • Renamed .github/workflows/azure-static-web-apps-agreeable-ground-01c3b230f.yml to .github/workflows/azure-static-web-apps.yml and updated the Azure Static Web Apps deployment steps to use the generic AZURE_STATIC_WEB_APPS_API_TOKEN secret instead of an environment-specific one. [1] [2]

Backend API configuration:

  • Changed the default axios base URL in src/app/api/agent.api.ts to use the REACT_APP_BACKEND_URL environment variable if set, otherwise defaulting to https://localhost:5001/api, making backend connectivity more flexible and environment-agnostic.

CHECK LIST

  • СI passed
  • Сode coverage >=95%
  • PR is reviewed manually again (to make sure you have 100% ready code)
  • All reviewers agreed to merge the PR
  • I've checked new feature as logged in and logged out user if needed
  • PR meets all conventions

@Vektor19 Vektor19 self-assigned this Dec 28, 2025
@Vektor19 Vektor19 added the Azure label Dec 28, 2025
@Vektor19 Vektor19 requested a review from shribakb1 December 28, 2025 21:43
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
2 Security Hotspots

See analysis details on SonarQube Cloud

@github-actions
Copy link
Copy Markdown

Azure Static Web Apps: Your stage site is ready! Visit it here: https://lively-tree-07bdde603-5.westeurope.4.azurestaticapps.net

@Vektor19 Vektor19 merged commit 5ac194d into main Dec 28, 2025
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants