-
Notifications
You must be signed in to change notification settings - Fork 36
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
Update Korean translation #75
base: feature
Are you sure you want to change the base?
Conversation
Is there a specific reason why it won't merge? |
I'm very hesitant about merging PRs updating translations to languages I don't speak, since I can't check myself if they are correct. They're also a bit out of scope for this fork, which mostly collects bugfixes and new features - translation updates would fit better in a more central place like upstream. I don't mind this PR being open here - that way people can use it for whatever they might need it for - but for now I can't really merge it, sorry. |
I think it would be better to merge it and get some feedback. |
If the reason is that the existing translation has a lot of mistranslations and untranslated parts, and you can't check the translation problem, it's better to delete it except for the languages you can manage. |
I wouldn't have translated it if I didn't think it would be reflected like this... |
for FR11.wx
Why not approve it first and see what users think? |
b54f508
to
a455e07
Compare
914ad0e
to
0e44a34
Compare
I'd like to contribute to making the tool more accessible to people with Korean accessibility.