-
Notifications
You must be signed in to change notification settings - Fork 0
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
Redirect potentially non Next Gen users to prototype version #247
Conversation
microbit-grace
commented
Feb 29, 2024
•
edited
Loading
edited
- Shown to users before browser compatibility warning dialog
- Shown only if they are detected to be in the UK (GB), Isle of Man (IM), Jersey (JE), or Guernsey (GG)
- Should only be shown once to the user
- API proxy for calling browser info endpoint for local dev via .env
![Screenshot 2024-02-29 at 14 48 41](https://private-user-images.githubusercontent.com/145345672/308965121-aa1a56fb-bd3f-4d07-9876-b0441d7d24e2.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA3OTAyOTQsIm5iZiI6MTcyMDc4OTk5NCwicGF0aCI6Ii8xNDUzNDU2NzIvMzA4OTY1MTIxLWFhMWE1NmZiLWJkM2YtNGQwNy05ODc2LWIwNDQxZDdkMjRlMi5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzEyJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcxMlQxMzEzMTRaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1hZTBiNWI0YWUyNWY1MDY0YzMzOGZhZTM5NzIxNDFkYjYzZTI4ZDc2NTk5Y2VhZjQ5OTFhZmI1YzhjZmJiOTJiJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.CQhDtg_0TKdCcjW211mUIo8RyaJInm5qB-8JdHBzeSw)
Preview build will be at |
209046f
to
21d30c5
Compare
…#243) - Shown to users before browser compatibility warning dialog - Shown only if they are detected to be in the UK (GB), Isle of Man (IM), Jersey (JE), or Guernsey (GG) - Should only be shown once to the user - API proxy for calling browser info endpoint for local dev via .env (cherry picked from commit 0aa4729)
21d30c5
to
35d0468
Compare
if (!get(hasSeenAppVersionRedirectDialog)) { | ||
const { country } = await fetchBrowserInfo(); | ||
const nextGenAvailableCountries = ['GB', 'JE', 'IM', 'GG']; | ||
isPotentiallyNonNextGenUser = !nextGenAvailableCountries.includes(country || ''); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I find it strange that TypeScript complains about country being undefined, as it's perfectly valid code without the || ''
, nevermind.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.