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

locale(ko): update Korean localization #1368

Merged
merged 9 commits into from
May 4, 2023
Merged

Conversation

ZISQO
Copy link
Contributor

@ZISQO ZISQO commented May 2, 2023

the korean language pack was different from the new version, so reworked based on the recently english language pack.
0 missing 0 unused keys with space 4, CRLF, UTF-8 with prettier on VSCode.

Signed-off-by: zisqo zisqofoto@gmail.com

ZISQO added 8 commits May 3, 2023 05:04
There was a json modification error due to the i18n conversion tool problem and it was attached as LF, not CRLF with UTF-8(prettier on VSCode. so i fixed up it and uploaded again.
this is just test for pr
There was a json modification error due to the i18n conversion tool problem and it was attached as LF, not CRLF with UTF-8(prettier on VSCode. so i fixed up it and uploaded again. (also remove 2 unused keys)
i don't know why stucked at prettier test when i make PR even if 0 missing and 0 unused key. also using UTF-8, CRLF, 4 space for tab with prettier on VSC
There was a json modification error due to the i18n conversion tool problem and it was attached as LF, not CRLF with UTF-8(prettier on VSCode. so i fixed up it and uploaded again. (also remove 2 unused keys)
The composition of the existing Korean language pack and the new English language pack is different, so the Korean language pack has been re-modified based on the English language pack. But I don't know why the first line error in the ko.json file occurs in the prettier check when i push the PR to develope.

i've check it again 0 missing key, 0 unused key, CRLF, UTF-8, 4 space, JSON format now.
let replace ko.json file with space 4 saved file
the korean language pack was different from the new version, so reworked based on the recently english language pack. 
0 missing 0 unused keys with space 4, CRLF, UTF-8 with prettier on VSCode.
@github-actions
Copy link
Contributor

github-actions bot commented May 2, 2023

Language file analysis report:

File Missing Keys Unused Keys
ko.json 0 0

@ZISQO ZISQO changed the title replace ko.json locale file fix: ko.json locale file May 2, 2023
@ZISQO ZISQO changed the title fix: ko.json locale file locale(ko-kr): new ko.json locale file May 2, 2023
@ZISQO ZISQO changed the title locale(ko-kr): new ko.json locale file fix: new ko.json locale file May 2, 2023
@dw-0 dw-0 changed the title fix: new ko.json locale file locale(ko): update Korean localization May 3, 2023
@dw-0
Copy link
Member

dw-0 commented May 3, 2023

I really have no idea what your usual workflow for creating PRs is, but i think it's not required to always create a new Pull Request if you change something. The pull request will update if you make changes to your local branch and push it.
image

Signed-off-by: Dominik Willner <th33xitus@gmail.com>
@github-actions
Copy link
Contributor

github-actions bot commented May 3, 2023

Language file analysis report:

File Missing Keys Unused Keys
ko.json 0 0

@ZISQO
Copy link
Contributor Author

ZISQO commented May 4, 2023

PR 생성을 위한 일반적인 워크플로우가 무엇인지 전혀 모르지만 무언가를 변경하면 항상 새로운 풀 리퀘스트를 생성할 필요는 없다고 생각합니다. 풀 요청은 로컬 브랜치를 변경하고 푸시하면 업데이트됩니다. 영상

I really have no idea what your usual workflow for creating PRs is, but i think it's not required to always create a new Pull Request if you change something. The pull request will update if you make changes to your local branch and push it. image

** ya i gotti! and thax to merge it of! : )

@meteyou meteyou merged commit 0e86652 into mainsail-crew:develop May 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants