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

Change translation error to warning to prevent user notifications. #321

Merged
merged 1 commit into from
Dec 20, 2024

Conversation

chase9
Copy link
Contributor

@chase9 chase9 commented Dec 15, 2024

Users were being shown a notification when translations didn't load, which includes when a translation was not needed (English). This change stops that.

This may solve #305

Users were being shown a notification when translations didn't load, which includes when a translation was not needed (English). This change stops that.
@l0drex
Copy link
Collaborator

l0drex commented Dec 17, 2024

Makes sense. Maybe even change the string to The app has not been translated to your langage yet.

@l0drex l0drex merged commit 3fcd817 into oskarsh:beta Dec 20, 2024
3 checks passed
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