Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Update Korean translation #8262

Merged
merged 3 commits into from
Jul 17, 2014
Merged

Update Korean translation #8262

merged 3 commits into from
Jul 17, 2014

Conversation

revi
Copy link
Contributor

@revi revi commented Jun 27, 2014

Add new translation, as well as making few changes (space, typo, etc...) to existing translations.

Note: I couldn't understand about SHA-1 things in here, so help is appreciated!

Revi added 2 commits June 27, 2014 16:44
Add new translation, as well as making few changes (space, typo, etc...) to existing translations.
@redmunds
Copy link
Contributor

@revi-wm We ask that community contributed language translations add this comment to the end of the strings.js file:

/* Last translated for 36fd23e311b2bb7f64c37e56c00006d8795050da */

The long string of hexadecimal chars (i.e. SHA) is an identifier that tells everyone which revision that your translation was done at so it's easier for the next update.

After you have added the comment:

  1. Follow this link to the root language strings.js file.
  2. Click on the "History" button in the upper-right corner of page
  3. Move mouse over the top (most recent) change and you'll see a clipboard appear on the right side. Move mouse over clipboard and click to copy the SHA
  4. Select SHA in comment and the paste to replace.

@revi
Copy link
Contributor Author

revi commented Jun 28, 2014

@redmunds Added SHA-1 at the bottom. Please check if this is correct.

@ds5apn
Copy link

ds5apn commented Jun 30, 2014

good improvement !

@redmunds
Copy link
Contributor

Triage Complete.

@ds5apn Did you review the changes here?

@redmunds redmunds self-assigned this Jul 17, 2014
@redmunds
Copy link
Contributor

Looks good. Merging.

redmunds added a commit that referenced this pull request Jul 17, 2014
Update Korean translation
@redmunds redmunds merged commit e37bd08 into adobe:master Jul 17, 2014
@revi revi mentioned this pull request May 25, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants