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

Add EN (US) locale fallback for when window.locale is null. #130

Merged
merged 2 commits into from
Mar 8, 2021

Conversation

Joran-Dob
Copy link
Contributor

Added an EN (US) locale fallback for when the window locale is null. Because the window.locales list is empty unset.

This currently throws an error on flutter 2.0.0 when building for web.

This is a fix for #129

Added a EN (US) locale fallback for when the window locale is null. Because the window.locales list is empty unset. 

This currently throws an error on flutter 2.0.0 when building for web.
Added ignore rule for dead_null_aware_expression.
@passsy
Copy link
Member

passsy commented Mar 8, 2021

Do you have a stacktrace for this? The lint dead_null_aware_expression makes clear that window.locale never returns null.
Please provide your flutter doctor -v output so we can reproduce this bug

@passsy
Copy link
Member

passsy commented Mar 8, 2021

I could confirm that bug on stable, but not on beta or dev. Thanks for the fix!

@passsy passsy merged commit 2b838c4 into wiredashio:dev Mar 8, 2021
@Joran-Dob
Copy link
Contributor Author

Great, thanks for the great product guys! 🎊

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