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

Added the Korean (kr) language support #2470

Open
wants to merge 2 commits into
base: v2-legacy
Choose a base branch
from

Conversation

WoonchanCho
Copy link
Contributor

Hi team,

Thank you for all your efforts for implementing this create viewer.

I want to add the the Korean locales to the viewer.

If you prefer letting me to work on the Locize, you can create the space for the Korean language and let me know too.

Thank you in advance for your support.

@codecov
Copy link

codecov bot commented Jul 11, 2021

Codecov Report

Merging #2470 (d6ce5e8) into master (d575e36) will not change coverage.
The diff coverage is n/a.

❗ Current head d6ce5e8 differs from pull request most recent head cbfa629. Consider uploading reports for the commit cbfa629 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2470   +/-   ##
=======================================
  Coverage   13.33%   13.33%           
=======================================
  Files         288      288           
  Lines        7664     7664           
  Branches     1465     1465           
=======================================
  Hits         1022     1022           
  Misses       5378     5378           
  Partials     1264     1264           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2afcd49...cbfa629. Read the comment docs.

Copy link
Member

@sedghi sedghi left a comment

Choose a reason for hiding this comment

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

I'm not familiar with the Korean language, but in Locize ko-KR seems to be the only option. Is this the same as kr that you have used?

We use Locize to maintain the translations. You can read more here.

Can you change your PR according to ko-KR, since all files will get overwritten by Locize ...

image

@@ -41,6 +41,13 @@ module.exports = (env, argv) => {
path: DIST_DIR,
filename: isProdBuild ? '[name].bundle.[chunkhash].js' : '[name].js',
publicPath: PUBLIC_URL, // Used by HtmlWebPackPlugin for asset prefix
devtoolModuleFilenameTemplate: function(info) {
Copy link
Member

Choose a reason for hiding this comment

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

why?

@@ -29,7 +29,7 @@ import OHIFDicomSegmentationExtension from '@ohif/extension-dicom-segmentation';
import OHIFDicomRtExtension from '@ohif/extension-dicom-rt';
import OHIFDicomMicroscopyExtension from '@ohif/extension-dicom-microscopy';
import OHIFDicomPDFExtension from '@ohif/extension-dicom-pdf';
//import OHIFDicomTagBrowserExtension from '@ohif/extension-dicom-tag-browser';
import OHIFDicomTagBrowserExtension from '@ohif/extension-dicom-tag-browser';
Copy link
Member

Choose a reason for hiding this comment

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

please limit this PR to be only for translation

@stale
Copy link

stale bot commented Jul 10, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the Stale 🥖 label Jul 10, 2022
@sedghi sedghi removed the Stale 🥖 label Jan 11, 2023
@sedghi sedghi changed the base branch from master to v2-legacy June 19, 2023 13:22
@sedghi
Copy link
Member

sedghi commented Jun 19, 2023

base has changed, read more here #3477

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v2-legacy OHIF v2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants