-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Korean translation #1083
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
Korean translation #1083
Conversation
Thanks @progh2, that's Awesome. 😄 Looking through the diff now, it seems very complete. Do you reckon it's ok to merge "as is" or should I take screenshots of everything first so you can look at them before it's merged? 😄 |
Please take screenshots.. |
Will do. It will probably be several hours from now (late tonight UK time) until I can start that as I'm focused on other stuff right now. Writing SQL queries and stuff at the moment... 😄 |
Sorry @progh2, haven't gotten this done yet. Need sleep. I'll get the screenshots done tomorrow. 😄 |
@progh2 Ok, I think that's screenshots for everything. 😄 Some strings don't seem correct yet:
Screenshots
|
Okay. |
I fixed the bug (*.sql vs. *.cvs). #1084
--> Many people often use it as an english word but translate it.
--> I can not find that phrase. Not yet applied to translation files? If I can see that, I'will translate it. |
Oops. Yep, my fault. I'll merge this PR now, then update the sqlb_ko_KR.ts file again. In theory, the only things which should change will be the bits for the Remote panel. 😄 |
I've just updated the source strings into a version sqlb_ko_KR.ts file. These seem like the only real changes: 😄
|
Thank you for waiting. =)