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

Add Korean translation #85

Merged
merged 1 commit into from
Aug 2, 2017
Merged

Add Korean translation #85

merged 1 commit into from
Aug 2, 2017

Conversation

miname
Copy link
Contributor

@miname miname commented Aug 2, 2017

I verified my work is based on develop branch

  • yes, keep going

Purpose of this Pull Request?

To make Korean users happy. 👍

Where should the reviewer start its review?

A new file titled as "contrast-finder_utf8_ko.properties" added.

How the PR could be tested?

No breaking changes, I hope.

Any background context you want to provide?

There are two paragraphs(line no. 51 and 57) where its usage is unknown. So I just left it untouched. Could you tell me in what context the phrase being used?

@mfaure
Copy link
Member

mfaure commented Aug 2, 2017

Wouaw, awesome! Great job @miname! Thank you very much. @dzc34 will surely integrate this in the next release.

dzc34 added a commit to dzc34/Contrast-Finder that referenced this pull request Aug 2, 2017
…legalArgumentException: unknown format type: 선택
dzc34 added a commit to dzc34/Contrast-Finder that referenced this pull request Aug 2, 2017
dzc34 added a commit to dzc34/Contrast-Finder that referenced this pull request Aug 2, 2017
dzc34 added a commit to dzc34/Contrast-Finder that referenced this pull request Aug 2, 2017
dzc34 added a commit to dzc34/Contrast-Finder that referenced this pull request Aug 2, 2017
dzc34 added a commit to dzc34/Contrast-Finder that referenced this pull request Aug 2, 2017
dzc34 added a commit to dzc34/Contrast-Finder that referenced this pull request Aug 2, 2017
@dzc34 dzc34 merged commit c7f1c60 into Asqatasun:develop Aug 2, 2017
dzc34 added a commit to dzc34/Contrast-Finder that referenced this pull request Aug 2, 2017
----------------------

  set version to 0.8.0
  updated CHANGELOG + typo
  fixed languages menu (Asqatasun#85 "Korean translation")
  i18n - fixed typo (Asqatasun#85 "Korean translation")
  i18n - added space before "\" at the end of the line (Asqatasun#85 "Korean translation")
  Fixed accessibility issue: added some lang="en" (Asqatasun#85 "Korean translation")
  Korean added to languages menu (Asqatasun#85 "Korean translation")
  fixed Asqatasun#86 "Korean translation" (Asqatasun#85) - java.lang.IllegalArgumentException: unknown format type: 선택
  Add Korean translation
  webapp - updated "Contribute to Contrast-Finder" (en + fr)
  Revert "Removed unused i18n entries"
  fix FR grammatical error
  fix Asqatasun#83
  Removed unused i18n entries
  docker/build_and_run-with-docker.sh (typo)
  docker/build_and_run-with-docker.sh - added new option (--log-build)
  updated docker/build_and_run-with-docker.sh (typo)
  renamed docker/build_and_run-with-docker.sh
  fixed .gitignore
  set version to 0.7.1-dev
@dzc34
Copy link
Member

dzc34 commented Aug 2, 2017

@miname, thank you very much. Integrated this in the new release.
https://app.contrast-finder.org/?lang=ko

@dzc34
Copy link
Member

dzc34 commented Aug 2, 2017

There are two paragraphs(line no. 51 and 57) where its usage is unknown. So I just left it untouched. Could you tell me in what context the phrase being used?

see #88

@miname miname deleted the i18n-korean branch August 3, 2017 00:10
@dzc34
Copy link
Member

dzc34 commented Aug 5, 2017

#88 is now fixed
and available in the last release
https://forum.asqatasun.org/t/contrast-finder-new-release-v0-8-1-korean-translation/357

@dzc34 dzc34 added the i18n label Apr 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants