-
Notifications
You must be signed in to change notification settings - Fork 6
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
Translate frontend with i18next #164
base: main
Are you sure you want to change the base?
Conversation
Thank you @alex-ahumada! Will review ASAP |
Thanks @dafeder! We should remove debug from i18next config file before release. |
Codeclimete warns about: That can't be fixed as i18next language config blocks are similar by design. |
@alex-ahumada I know its been a while on these PRs but we do intend to get back to them soon. Not sure if you're able to update them so they merge cleanly again; I imagine your fork has diverged a bit but that would be helpful. I'm sure we can override that codeclimate gate if that's the only blocker 😄 |
I'll try update the fork and resolve the conflicts in the coming days @dafeder ! |
Frontend translation using i18next , with browser language detection and EN/ES translations.
We should probably move translations from config file to individual language files when more languages are added.
resolves #78