-
-
Notifications
You must be signed in to change notification settings - Fork 391
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
Newsletter localization #4485
Newsletter localization #4485
Conversation
Let's be consistent with Emby and Jellyfin
I worked on this before the refactoring was merged to develop, sorry for the mess in the commit list. The relevant commits are near the bottom (from 45c5c1e). |
That's fine, they all get squashed before going to develop anyway. I'm going to pull this one down and do a bit of testing, but looking over the code no complaints :) There are some failing tests mind. Actually the failing tests are flakey tests... I'll fix them |
This will allow some strings in the newsletter to be translated through the use of .resx files. These files are supported by Crowdin and could be added there to allow for global localization. I've already translated the FR part for testing purposes and included it in this PR.
For now the newsletter will use the global language settings, it won't take into account the user's individual preferences.
It will use English as default if there is no resource file present for the set language, so it should be safe to deploy before integrating it into Crowdin and have the generated files for every language.
I hope I did right by the guidelines with the I18n packaging, this is fairly new to me. I'm open to feedback.