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 localization #2504

Merged
merged 1 commit into from
Feb 17, 2024
Merged

Conversation

zooxop
Copy link
Contributor

@zooxop zooxop commented Feb 15, 2024

Hello!

First of all, I would like to thank the Sparkle project for the amazing feature support.
It made it very easy for me to implement the ability to update my app.

Anyway, while applying Sparkle to my app, I noticed that some Korean localization was missing.
So, I added the missing Korean localization keys and fixed some other expressions that I personally felt were unnatural.

Misc Checklist

  • My change requires a documentation update on Sparkle's website repository
  • My change requires changes to generate_appcast, generate_keys, or sign_update

Testing

I tested and verified my change by using one or multiple of these methods:

  • Sparkle Test App
  • Unit Tests
  • My own app
  • Other (please specify)

What I tested :

  • <sparkle:criticalUpdate>
  • <sparkle:minimumSystemVersion>
  • <sparkle:maximumSystemVersion>

macOS version tested: 14.1

@@ -4,17 +4,20 @@
/* Class = "NSTextFieldCell"; title = "Text Cell"; ObjectID = "45"; */
"45.title" = "Text Cell";

/* Class = "NSTextFieldCell"; title = "Anonymous system profile information is used to help us plan future development work. Please contact us if you have any questions about this.\n\nThis is the information that would be sent:"; ObjectID = "183"; */
"183.title" = "익명으로 보내지는 시스템 정보로 차후 프로그램 개발에 도움이 될 수 있습니다. 질문이 있으시면 연락 주십시오.\n\n아래 정보가 전송될 것입니다.";
/* Class = "NSButtonCell"; title = "Check Automatically"; ObjectID = "176"; */
Copy link
Member

Choose a reason for hiding this comment

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

It doesn't matter a lot but the ObjectID should stay as OhZ-1K-DmA. I guess the English one is wrong.


/* Class = "NSButtonCell"; title = "Don’t Check"; ObjectID = "cCJ-V0-aTi"; */
Copy link
Member

@zorgiepoo zorgiepoo Feb 17, 2024

Choose a reason for hiding this comment

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

Same here, this ObjectID was right.

@zorgiepoo
Copy link
Member

Had some minor comments that doesn't really affect anything. Looks good to me. Thanks!

@zorgiepoo zorgiepoo merged commit 24f0f39 into sparkle-project:2.x Feb 17, 2024
2 checks passed
@zorgiepoo zorgiepoo added this to the 2.6 milestone Feb 17, 2024
@zooxop
Copy link
Contributor Author

zooxop commented Feb 17, 2024

Had some minor comments that doesn't really affect anything. Looks good to me. Thanks!

To avoid mistakes, I copied and pasted the following code base for the work, and I synchronized the comments in the process.

/* Class = "NSButtonCell"; title = "Check Automatically"; ObjectID = "176"; */
"OhZ-1K-DmA.title" = "Check Automatically";
/* Class = "NSButtonCell"; title = "Don’t Check"; ObjectID = "177"; */
"cCJ-V0-aTi.title" = "Don’t Check";

I didn't sufficiently consider the possibility that this modification might be wrong!

It's fortunate that it doesn't matter much.
Thank you for your thorough review!

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.

2 participants