Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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 pallets-eco:master Feb 14, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants