Skip to content
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

Added missing Korean translations. #732

Merged
merged 1 commit into from Feb 14, 2022
Merged

Added missing Korean translations. #732

merged 1 commit into from Feb 14, 2022

Conversation

josueisonfire
Copy link
Contributor

I have added the missing Korean translations, though I have to ask, the translation for the string,
"Choices cannot be None."
is a bit tricky, due to None being a Python programming term, and there is no direct translation for it (the same applies for null). Korean programmers will most likely understand if I refer to it as None, so I left it that way.
If "Choices cannot be None. " is asking the user that an empty field is invalid, the transaltion
"선택 값이 없을 수 없습니다." (meaning 'the choice cannot be empty') is more appropriate.

I have added the missing Korean translations, though I have to ask, the translation for the string, 
```"Choices cannot be None."``` 
is a bit tricky, due to `None` being a Python programming term, and there is no direct translation for it (the same applies for null). Korean programmers will most likely understand if I refer to it as `None`, so I left it that way. If 
```"Choices cannot be None. "```
 is asking the user that an empty field is invalid, the transaltion
```"선택값이 없을 수 없습니다."```
(meaning that ) is more appropriate.
@josueisonfire josueisonfire changed the title Update wtforms.po Added missing Korean translations. Feb 9, 2022
@azmeuk
Copy link
Member

azmeuk commented Feb 12, 2022

Hi. Thank you for you contribution. I cannot really help to review korean. Is this patch OK to merge or do you prefer to be sure of your translation first?

@josueisonfire
Copy link
Contributor Author

Yes, I think the translation is good as it is.
Sorry for the confusion, and thanks!

@azmeuk azmeuk merged commit 08aca67 into wtforms:master Feb 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants