Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Let you choose which locale to use on the sandbox #2044

Open
thewilkybarkid opened this issue Oct 25, 2024 · 0 comments
Open

Let you choose which locale to use on the sandbox #2044

thewilkybarkid opened this issue Oct 25, 2024 · 0 comments

Comments

@thewilkybarkid
Copy link
Member

No description provided.

erkannt added a commit that referenced this issue Oct 25, 2024
thewilkybarkid added a commit that referenced this issue Oct 25, 2024
This change sets the locale for the request based on a cookie. There's no way to set the cookie through the UI, and the only valid option is the DefaultLocale anyhow so it's safe to include without a feature flag.

Refs #2044
thewilkybarkid added a commit that referenced this issue Oct 25, 2024
This change avoids duplicating reading the cookie on the legacy pages by passing through the already-read value. This requires a change to our Express usage to allow for request-specific arguments. As we're not using Express to listen for requests, we've favoured this type-safe way (as opposed to Express's standard of modifying the request object).

Refs #2044
erkannt added a commit that referenced this issue Oct 25, 2024
thewilkybarkid added a commit that referenced this issue Oct 25, 2024
This is never accessible as it's behind a feature flag, and doesn't work either.

Refs #2044
erkannt added a commit that referenced this issue Oct 25, 2024
thewilkybarkid added a commit that referenced this issue Oct 25, 2024
Note this only works on Express pages, not Effect pages.

Refs #2044
erkannt added a commit that referenced this issue Oct 25, 2024
Ensure locale cookie only is used when feature flag is enabled.

Refs: #2044
erkannt added a commit that referenced this issue Oct 25, 2024
erkannt added a commit that referenced this issue Oct 25, 2024
This is limited to supported locales which currently only is 'en-US'.

Refs: #2044
erkannt added a commit that referenced this issue Oct 25, 2024
Currently only one strings is translated, the main CTA on the homepage.

Refs: #2044
thewilkybarkid added a commit that referenced this issue Oct 25, 2024
thewilkybarkid added a commit that referenced this issue Oct 25, 2024
thewilkybarkid added a commit that referenced this issue Oct 25, 2024
thewilkybarkid added a commit that referenced this issue Oct 25, 2024
thewilkybarkid added a commit that referenced this issue Oct 25, 2024
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

No branches or pull requests

1 participant