Skip to content

update hosting urls#1779

Merged
Alek99 merged 1 commit intomainfrom
carlos/update-deploy-ci
Mar 17, 2026
Merged

update hosting urls#1779
Alek99 merged 1 commit intomainfrom
carlos/update-deploy-ci

Conversation

@carlosabadia
Copy link
Copy Markdown
Collaborator

No description provided.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Mar 16, 2026

Greptile Summary

Updates the hosting URLs in dev and staging CI/CD workflows, migrating from the old rxc.app/reflexcorp.run/fly.dev domains to the new flexgen-dev.reflex.dev and flexgen-staging.reflex.dev domains.

  • Both CP_WEB_URL and CP_BACKEND_URL now point to the same URL in each environment (previously they were separate frontend and backend hosts)
  • The dead link checker in deploy-dev.yml (line 57) still references the old rxc.app domain — may need a follow-up update
  • FLY_API_TOKEN remains in deploy-stg.yml but may be unused now that the URLs no longer point to Fly.dev

Confidence Score: 4/5

  • This PR is safe to merge — it's a straightforward URL update in CI config with low risk of breakage.
  • The changes are minimal and well-scoped: only environment variable URL values are updated in two workflow files. The only concerns are the now-identical web/backend URLs (likely intentional) and minor cleanup opportunities (dead link checker URL, unused FLY_API_TOKEN).
  • No files require special attention, though .github/workflows/deploy-dev.yml line 57 (dead link checker) still references the old domain.

Important Files Changed

Filename Overview
.github/workflows/deploy-dev.yml Updates CP_WEB_URL and CP_BACKEND_URL from old rxc.app/fly.dev domains to flexgen-dev.reflex.dev. Both variables now point to the same URL. Note: dead link checker on line 57 still uses the old rxc.app domain.
.github/workflows/deploy-stg.yml Updates CP_WEB_URL and CP_BACKEND_URL from old reflexcorp.run/fly.dev domains to flexgen-staging.reflex.dev. Both variables now point to the same URL. FLY_API_TOKEN env var remains but may be unused now that URLs no longer point to Fly.dev.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Push to main branch] --> B[deploy-dev.yml]
    C[Push to r/* branch] --> D[deploy-stg.yml]
    
    B --> E["CP_WEB_URL: flexgen-dev.reflex.dev"]
    B --> F["CP_BACKEND_URL: flexgen-dev.reflex.dev"]
    
    D --> G["CP_WEB_URL: flexgen-staging.reflex.dev"]
    D --> H["CP_BACKEND_URL: flexgen-staging.reflex.dev"]
    
    E --> I[reflex deploy to dev]
    F --> I
    G --> J[reflex deploy to staging]
    H --> J
    
    I --> K[Dead link checker still uses old rxc.app URL]
Loading

Last reviewed commit: d264c77

@carlosabadia carlosabadia marked this pull request as draft March 16, 2026 17:14
@carlosabadia carlosabadia marked this pull request as ready for review March 17, 2026 19:33
@Alek99 Alek99 merged commit e466cd3 into main Mar 17, 2026
10 checks passed
@Alek99 Alek99 deleted the carlos/update-deploy-ci branch March 17, 2026 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants