-
Notifications
You must be signed in to change notification settings - Fork 2
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 translations #787
add translations #787
Conversation
@aaron-collier I tried to rebase this but I think I added some changes that weren't supposed to be added. I don't want to make it worse. Can you fix this please? |
6f36593
to
c2dc6b1
Compare
4c6ffbe
to
859a230
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.
a couple questions about changes to the translation maps, not sure if typos in translation maps or if typos upstream being accounted for by translation maps. and one question about a removed TODO that seemed like maybe it should've stayed.
the "request changes" is more about the TODO, since i wasn't sure whether i'd noticed an actual problem with the translations.
Why was this change made?
How was this change tested?
Which documentation and/or configurations were updated?