-
Notifications
You must be signed in to change notification settings - Fork 8
Combine internationalization #158
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
Combine internationalization #158
Conversation
@@ -0,0 +1,59 @@ | |||
{ |
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.
Did you write these strings? Do you speak Spanish?
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.
I had Copilot (Claude 4 Sonnet) translate for me.
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.
Can you add a comment that says we should have a native Spanish speaker check these?
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.
Since JSON doesn't allow comments, I put a key:value pair at the top that was "TODO" and a note. How is that?
This actually puts two languages in the menu system and makes it work across React and Blockly. I haven't combed through the code yet looking for all strings and changing them.