-
Notifications
You must be signed in to change notification settings - Fork 0
feat: add localization #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
Conversation
✅ Deploy Preview for sovryn-layer ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
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.
Pull Request Overview
This PR adds internationalization (i18n) support to the web application using react-i18next. It sets up the necessary infrastructure for localization including configuration, translation files, and a demo route.
Key changes:
- Adds i18next dependencies and configuration with HTTP backend for loading translation files
- Creates English translation files for common, glossary, and validation namespaces
- Adds a demo route to showcase translation functionality
Reviewed Changes
Copilot reviewed 11 out of 13 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| apps/web-app/vite.config.ts | Adds i18next Vite plugin configuration and updates path imports |
| apps/web-app/tsconfig.json | Includes custom types for i18n type definitions |
| apps/web-app/src/routes/i18n/index.tsx | Demo route showcasing translation usage with type-safe selectors |
| apps/web-app/src/routeTree.gen.ts | Generated route tree updates for new i18n route |
| apps/web-app/src/main.tsx | Adds i18n initialization and Suspense wrapper for async loading |
| apps/web-app/src/i18n/index.ts | Core i18n configuration with backend, language detection, and React integration |
| apps/web-app/public/locales/en/*.json | English translation files for different namespaces |
| apps/web-app/package.json | Adds required i18n dependencies |
| apps/web-app/.gitignore | Excludes generated TypeScript definition files for locales |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
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.
Pull Request Overview
Copilot reviewed 11 out of 13 changed files in this pull request and generated no new comments.
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
ed223d8 to
3472f68
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.
Pull Request Overview
Copilot reviewed 11 out of 13 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
No description provided.