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

Update Korean Translation #2000

Merged
merged 3 commits into from
Oct 19, 2023
Merged

Update Korean Translation #2000

merged 3 commits into from
Oct 19, 2023

Conversation

clairecharles
Copy link
Contributor

Description

Added the Korean translations to the untranslated string

Motivation and Context

Fixes #1988

@clairecharles
Copy link
Contributor Author

Hi @frankiejol, I translated the Korean file but the GitHub checks have been failing and i don't know why. Could you please look into it?

@frankiejol frankiejol self-requested a review October 16, 2023 06:39
Copy link
Member

@frankiejol frankiejol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First of all thank you very much for your submission, it looks amazing.
Then there are a couple of errors, it looks like at line 337, the " is missing at the end of the string, but there are two of them at the beginning. Similar errors are reported by the automatic check at lines: 394, 425 and 880

@frankiejol frankiejol added hacktoberfest localization translation and localization labels Oct 16, 2023
@clairecharles
Copy link
Contributor Author

Hi @frankiejol, thank you all the checks have passed. Can you please review it?

Copy link
Collaborator

@amparorvd amparorvd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the correction

@frankiejol frankiejol added this to the 2.2 milestone Oct 19, 2023
@frankiejol frankiejol merged commit 40fe94c into UPC:main Oct 19, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Translations update
3 participants