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

Add internationalization and replace all hardcoded labels with i18n entries #211

Closed
wants to merge 5 commits into from

Conversation

ManyRios
Copy link
Contributor

@ManyRios ManyRios commented Aug 30, 2024

Describe changes

Added react-i18next dependency for packages/features.
Added all necessary config files for the i18next library.
Extracted all necessary i18n entries from hardcoded labels in Pallad to locales/en.json

Ticket or discussion link

#194

All the labels has been changed in packages/features.

Copy link

deepsource-io bot commented Aug 30, 2024

Here's the code health analysis summary for commits 1b34dba..c19af51. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ Success
❗ 2 occurences introduced
🎯 4 occurences resolved
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@ManyRios
Copy link
Contributor Author

@mrcnk Please review...

@mrcnk
Copy link
Member

mrcnk commented Aug 30, 2024

Hi @ManyRios, the wallet doesn't seem to work when I built it:
CleanShot 2024-08-30 at 21 53 49@2x

@ManyRios
Copy link
Contributor Author

@mrcnk I dont have conflicts... please check again
image

@ManyRios
Copy link
Contributor Author

ManyRios commented Sep 3, 2024

Hey @mrcnk please review this PR

@ManyRios ManyRios closed this Oct 17, 2024
@ManyRios ManyRios deleted the i18next branch October 17, 2024 02:18
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