-
-
Notifications
You must be signed in to change notification settings - Fork 55
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
client: Spanish Localization #1263
Conversation
Victor-M-Giraldo
commented
Jan 23, 2024
•
edited
Loading
edited
- Pull requests adds localization in Spanish to OTT.
WalkthroughThe recent update introduces a Spanish language option to the application's navigation bar, allowing users to select Spanish as their interface language. This enhancement is accompanied by the necessary Spanish translations for the user interface, ensuring a seamless user experience for Spanish-speaking users. Changes
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
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.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files selected for processing (2)
- client/src/components/navbar/LocaleSelector.vue (1 hunks)
- client/src/locales/sp.ts (1 hunks)
Additional comments: 2
client/src/components/navbar/LocaleSelector.vue (1)
- 33-36: Added a new language option for Spanish with the flag "🇪🇸" and the value "sp" to the list of locales in the
LocaleSelector
component.client/src/locales/sp.ts (1)
- 1-378: The Spanish localization file
sp.ts
has been added, containing translations for various UI elements and messages.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1263 +/- ##
================================================
+ Coverage 61.5425% 61.5711% +0.0285%
================================================
Files 117 117
Lines 9413 9420 +7
Branches 1155 1158 +3
================================================
+ Hits 5793 5800 +7
Misses 3620 3620 ☔ View full report in Codecov by Sentry. |
Seeing lots of errors for the lint. Could it be the upside down question marks/exclamation points, or letters with accents on them? Update: It's good now, nevermind. |
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.
Looks good, just needs a minor fix
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.
Review Status
Actionable comments generated: 15
Configuration used: CodeRabbit UI
Files selected for processing (2)
- client/src/components/navbar/LocaleSelector.vue (1 hunks)
- client/src/locales/es.ts (1 hunks)
Files skipped from review as they are similar to previous changes (1)
- client/src/components/navbar/LocaleSelector.vue
Additional comments: 2
client/src/locales/es.ts (2)
- 1-1: Ensure that the imported types (
OttWebsocketError
,BehaviorOption
,Role
) are used within this file, otherwise, the import is unnecessary.- 3-383: Check for consistency in translation quality, appropriate context, and correct spelling. Verify that the translations are accurate and make sense within the application's context.
Ah forgot a few. I'll change those. |
I did run yarn lint, it's giving me errors on my desktop though. I'll run it on my laptop and push again in a second. |
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.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files selected for processing (1)
- client/src/locales/es.ts (1 hunks)
Files skipped from review as they are similar to previous changes (1)
- client/src/locales/es.ts