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

Second translate kr #69

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

youjinup
Copy link

Second translate kr

Issue: #29 Korean (ko-KR) translation

I only uploaded the Excel file. Please check it out

@youjinup
Copy link
Author

And I received the project, but it doesn't run when I press .exe file. Can you tell me how to do it??

@arthurits
Copy link
Owner

arthurits commented Nov 18, 2023

Hi @youjinup!

Thanks for your time and work! I'm afraid this PR still attempts to modify all the files in the repo!
I believe you should delete First translate kr, which is the one commit causing all this mess. That commit is in the Second_Translate_kr branch in youjinup/SignalAnalysis and I can't delete it.

Both Second translate kr and Second Translate kr only contain modifications to the Excel file, so these two are fine.
image

And I received the project, but it doesn't run when I press .exe file. Can you tell me how to do it??

I'm not sure what the problem is. If you downloaded the software via the releases section from the project's github repo, then your PC needs to have the .NET SDK 7.0 installed in order to successfully run the .exe; that's be the only dependency to execute the software. If you already use Visual Studio with the Desktop Development environment installed, then the .NET SDK is most likely already installed, and you should experience no problems whatsoever running the software.

Here's a screenshot showing the partially translated main window. Unless I'm mistaken, your Excel file contained just 68 translated strings out of the 198 strings.
image

@youjinup
Copy link
Author

WOW Thank you!! I didn't know because it was my first time repo, but thank you for answering and letting me know. I'll do everything that hasn't been translated yet. Thank you for accepting my request.

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