feat(frontend): add cloud region selector on sign-in#3629
Merged
mmabrouk merged 10 commits intorelease/v0.85.4from Feb 12, 2026
Merged
feat(frontend): add cloud region selector on sign-in#3629mmabrouk merged 10 commits intorelease/v0.85.4from
mmabrouk merged 10 commits intorelease/v0.85.4from
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
dosubot
bot
added
the
size:L
Member
Author
|
@jp-agenta we need to set these Runtime env (secrets/vars): AGENTA_CLOUD_REGION=eu|us (and probably in staging or so for testing) |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
Contributor
However, in this PR, plenty of things seem to be hardcoded:
I'd say that adding an env var for the region only is not clean. Either we put everything in env vars or nothing. In this case, I'd
I guess the former is simpler. Now, in self-hosted, all this would still be visible ? Even the decision to show this whole thing or not follows the same rule:
|
- Remove AGENTA_CLOUD_REGION env var; region is now derived purely from window.location.hostname (the hostname is the source of truth) - Revert unrelated next.config.ts changes (bundleAnalyzer, duplicate transpilePackages, optimizePackageImports)
- Centralize all hostname logic into CLOUD_ENVIRONMENTS table - Add preview/staging envs (staging.preview.agenta.dev=EU, testing.preview.agenta.dev=US) - Add redirect-loop guard via sessionStorage - Fix ARIA: use radiogroup + radio + aria-checked - Fix hardcoded #586673 colors to theme tokens - Extract RegionButton as a proper component - Wrap export in error boundary (auth page won't crash) - Fix cookie parsing (substring instead of split) - Drop useless useMemo on isAuthCallback - Add aria-haspopup on Learn more button - Derive modal region list from REGIONS data
junaway
approved these changes
Feb 10, 2026
ardaerzin
requested changes
Feb 11, 2026
web/oss/src/components/pages/auth/RegionSelector/RegionInfoModal.tsx
Outdated
Show resolved
Hide resolved
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Config
AGENTA_CLOUD_REGION=eu|us(injected intoNEXT_PUBLIC_AGENTA_CLOUD_REGIONviaweb/entrypoint.sh)AGENTA_WEB_URL(setsNEXT_PUBLIC_AGENTA_WEB_URL),AGENTA_API_URL