-
Notifications
You must be signed in to change notification settings - Fork 21
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
Adding localization #445
base: main
Are you sure you want to change the base?
Adding localization #445
Conversation
Future998
commented
Oct 25, 2024
- Replaced the text for localization.
- Added Select to select the language in the Header.
- Localization of the DateTime format.
- Added Russian language.
@Future998 thanks for the PR. |
@Future998 seems to be working nicely. Can you pls rebase and fix the merge conflicts? |
856bca3
to
8f170a3
Compare
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
@Future998 pls address the ci failures. |
3dd802a
to
0eef473
Compare
@rchincha, OK, it's ready |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #445 +/- ##
==========================================
- Coverage 85.00% 84.25% -0.76%
==========================================
Files 63 64 +1
Lines 1961 2039 +78
Branches 532 541 +9
==========================================
+ Hits 1667 1718 +51
- Misses 284 308 +24
- Partials 10 13 +3 ☔ View full report in Codecov by Sentry. |
I don't quite understand what to do next? |
4f37f4e
to
73b21a5
Compare
@rchincha, OK, it's ready |
@Future998 This PR is a little late for the very next release. Will consider this for the next release after. |
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.
Thank you for the PR. The changes look good to me. If we want to add localisation, this will also set the basis for implementing other locales.
|
||
const { t, i18n } = useTranslation(); | ||
const [selectedLanguage, setSelectedLanguage] = useState(i18n.language); | ||
const handleLanguageChange = (event) => { |
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.
A test with language selection would be good to have, just to have it covered as it would be a new feature
- Replaced the text for localization. - Added Select to select the language in the Header. - Localization of the DateTime format. - Added Russian language. Signed-off-by: Alexander Burmatov <[email protected]>
The translation of the application affected the tests. Signed-off-by: Alexander Burmatov <[email protected]>
Signed-off-by: Alexander Burmatov <[email protected]>
73b21a5
to
384723a
Compare
Like that? |