-
Notifications
You must be signed in to change notification settings - Fork 3
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
Ot.language options #319
Ot.language options #319
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
key: 'es', | ||
label: 'Español', | ||
}, | ||
]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will we be easily able to add other languages to this array? Does this just use the i18 language library/autotranslate?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep! As long we have the translations for other languages, we can just add their key and appropriate label for languages here. We'll just need to make sure that the key we specify here matches what we use in src/i18n/i18n.ts
where we map languages to translation files.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great for English and Spanish, just refer to the question about other languages
Checklist
yarn run check
yarn run test
Why
Resolves #234
Provides a dropdown button for users to switch languages (if available)
This PR
removed FlexBar component from mobileNavBar, replaced with Flex. There is still some weird alignment issue with the button, so that needs to be fixed
Screenshots
Verification Steps
Ran the frontend, changed languages successfully