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

Hotfix to fix issue with wrong default locale causing Thunder to display in RTL #1039

Merged
merged 2 commits into from
Jan 11, 2024

Conversation

hjiangsu
Copy link
Member

@hjiangsu hjiangsu commented Jan 11, 2024

Pull Request Description

This is a hotfix PR which addresses the issue where the "Arabic" language is set by default. This causes Thunder to be RTL when it is not intended. This hotfix targets the main branch so that I can push out this fix for Google Play/App Store

Unfortunately, I'm unable to test this myself as I cannot reproduce the issue on my end. However, it should be a relatively safe change as it just applies the default en locale when none is present.

Issue Being Fixed

Issue Number: #1024, #929

Screenshots / Recordings

Checklist

  • Did you update CHANGELOG.md?
  • Did you use localized strings where applicable?
  • Did you add semanticLabels where applicable for accessibility?

@hjiangsu hjiangsu changed the title Hotfix to fix issue with wrong default locale Hotfix to fix issue with wrong default locale causing Thunder to display in RTL Jan 11, 2024
Copy link
Member

@micahmo micahmo left a comment

Choose a reason for hiding this comment

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

LGTM

@hjiangsu hjiangsu merged commit 6439e77 into main Jan 11, 2024
@hjiangsu hjiangsu deleted the hotfix/default-locale branch January 11, 2024 18:21
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