Skip to content

update RX_BUILD_BACKEND#1783

Merged
carlosabadia merged 1 commit intomainfrom
update-RX_BUILD_BACKEND
Mar 18, 2026
Merged

update RX_BUILD_BACKEND#1783
carlosabadia merged 1 commit intomainfrom
update-RX_BUILD_BACKEND

Conversation

@adhami3310
Copy link
Copy Markdown
Member

No description provided.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Mar 18, 2026

Greptile Summary

This PR makes two changes to pcweb/constants.py: it removes the unused RX_CLOUD_BACKEND constant and updates the default value of RX_BUILD_BACKEND from https://build-backend.reflex.dev/ to https://build.reflex.dev/api/. The new URL aligns with the existing REFLEX_CLOUD_URL and REFLEX_BUILD_URL constants already pointing to build.reflex.dev, consolidating the domain used for Reflex build infrastructure.

  • RX_CLOUD_BACKEND is confirmed to be unused across the codebase — its removal is safe.
  • RX_BUILD_BACKEND is consumed in pcweb/pages/landing/views/hero.py where it is used as the base for the /prompt API call (RX_BUILD_BACKEND.rstrip("/") + "/prompt"); the new default resolves to https://build.reflex.dev/api/prompt, which is the intended endpoint after this change.
  • The section comment # Reflex Cloud Backend is now stale and should be updated to # Reflex Build Backend.

Confidence Score: 5/5

  • This PR is safe to merge — it removes an unused constant and updates a backend URL default value with no logic changes.
  • The change is minimal: one constant is deleted (confirmed unused via codebase search) and one URL default is updated. The call site in hero.py handles trailing slashes correctly with rstrip("/"), so the new /api/ suffix in the default URL is handled cleanly. No logic, tests, or other files are affected.
  • No files require special attention.

Important Files Changed

Filename Overview
pcweb/constants.py Removes the unused RX_CLOUD_BACKEND constant and updates the default URL for RX_BUILD_BACKEND from https://build-backend.reflex.dev/ to https://build.reflex.dev/api/. The section comment # Reflex Cloud Backend is now stale since only the build backend constant remains.

Last reviewed commit: "update RX_BUILD_BACK..."

Comment on lines 81 to +82
# Reflex Cloud Backend
RX_CLOUD_BACKEND = os.getenv("RX_CLOUD_BACKEND", "https://cloud-backend.reflex.dev/")
RX_BUILD_BACKEND = os.getenv("RX_BUILD_BACKEND", "https://build-backend.reflex.dev/")
RX_BUILD_BACKEND = os.getenv("RX_BUILD_BACKEND", "https://build.reflex.dev/api/")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Stale section comment

The section comment # Reflex Cloud Backend is now inaccurate — RX_CLOUD_BACKEND has been removed and only RX_BUILD_BACKEND remains. The comment should be updated to reflect what the section actually contains.

Suggested change
# Reflex Cloud Backend
RX_CLOUD_BACKEND = os.getenv("RX_CLOUD_BACKEND", "https://cloud-backend.reflex.dev/")
RX_BUILD_BACKEND = os.getenv("RX_BUILD_BACKEND", "https://build-backend.reflex.dev/")
RX_BUILD_BACKEND = os.getenv("RX_BUILD_BACKEND", "https://build.reflex.dev/api/")
# Reflex Build Backend
RX_BUILD_BACKEND = os.getenv("RX_BUILD_BACKEND", "https://build.reflex.dev/api/")

@carlosabadia carlosabadia merged commit 222f7e5 into main Mar 18, 2026
10 checks passed
@carlosabadia carlosabadia deleted the update-RX_BUILD_BACKEND branch March 18, 2026 15:07
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