moaw icon indicating copy to clipboard operation
moaw copied to clipboard

Add Japanese translation to the GitHub Copilot workshop

Open SIkebe opened this issue 2 years ago • 6 comments

This PR introduces a Japanese translation for the GitHub Copilot workshop.

There’s one point that needs clarification: For the translated filename, should we use the language code such as ja, or the country code like jp? The CONTRIBUTING.md states:

Add translations

You can localize any workshop in this repository by creating a new folder translations in the same directory as the original folder, and naming any translated file <original_name>.<country_code>.<extension>. For example, the French translation of workshop.md would be translations/workshop.fr.md.

You can follow the same rule if you need to localize any asset, for example assets/image.png should be localized to assets/translations/image.fr.png.

However, I’ve noticed that various documents in this repository suggest using the language code. Could you please confirm which is correct?

SIkebe avatar Apr 16 '24 07:04 SIkebe

FYI, I published GitHub Pages so that you can review the translated workshop page. https://sikebe.github.io/moaw/workshop/github-copilot/translations/workshop.ja.md?step=0

SIkebe avatar Apr 18 '24 02:04 SIkebe

Thank you! Using language code here is fine. If the original assets (images/code) were not updated, could you update the links to point to the original one instead of duplicating them inside the /translations/ subfolder?

sinedied avatar Apr 22 '24 09:04 sinedied

Using language code here is fine.

Then, should I fix the CONTRIBUTING.md?

If the original assets (images/code) were not updated, could you update the links to point to the original one instead of duplicating them inside the /translations/ subfolder?

When I use relative links, the assets were not displayed and you can't download the assets zip file. So I copied those assets into the /translations/ subfolder. http://localhost:4200/workshop/github-copilot/translations/workshop.ja.md?step=0

Which url should we use to show the assets on translated page locally and on GitHub Page?

image

image

SIkebe avatar Apr 24 '24 05:04 SIkebe

Hi @sinedied Do you have any opinions or advice regarding the above issue?

SIkebe avatar May 08 '24 02:05 SIkebe

Hi @SIkebe and thank you for your patience. You're right it seems there's an issue when using ../assets, I'll open an issue and fix it as soon as possible. Keep in mind it may take 1-2 weeks as it's a very busy period, again I thank you for your contribution and patience.

sinedied avatar May 13 '24 08:05 sinedied

Thank you for confirming! Once the issue is resolved, I'll check that again.

SIkebe avatar May 13 '24 08:05 SIkebe

Hi @sinedied , what's the status of the issue?

SIkebe avatar Jul 05 '24 04:07 SIkebe

Thanks for your patience, the fix has been published and you can now reference assets from the parent folder.

sinedied avatar Jul 16 '24 09:07 sinedied

@sinedied Thanks! I've confirmed that issue had been fixed, so I removed the duplicated assets.

SIkebe avatar Jul 17 '24 00:07 SIkebe