FreeOTPPlus icon indicating copy to clipboard operation
FreeOTPPlus copied to clipboard

Please add Chinese Traditional translations

Open s8321414 opened this issue 3 years ago • 2 comments

Please add Chinese Traditional translations on Crowdin, I can contribute to it :smiley:

s8321414 avatar Jul 13 '22 23:07 s8321414

https://crowdin.com/project/freeotpplus/zh-TW

6543 avatar Jul 14 '22 00:07 6543

https://crowdin.com/project/freeotpplus/zh-TW

Thank you!

s8321414 avatar Jul 14 '22 07:07 s8321414

@6543 I noticed an issue in the Chinese translation. Both zh-CN and zh-TW are mapped to "zh" directory. And the PR overwrote the zh-CN with zh-TW instead of creating two different translations zh-CN for simplified Chinese and zh-TW for traditional Chinese. Can we fix this? Thanks! https://github.com/helloworld1/FreeOTPPlus/commit/893b9166195085253c2b7399e8d949c50759405a https://crowdin.com/project/freeotpplus/zh-CN https://crowdin.com/project/freeotpplus/zh-TW

helloworld1 avatar Nov 21 '22 19:11 helloworld1

well we either make one default "zh" or just migrate to use long names for all ...

6543 avatar Nov 22 '22 01:11 6543

@6543 Is it possible to have mix and match, or we have to use long name for all?

helloworld1 avatar Nov 22 '22 02:11 helloworld1

Version 3.0 is released. However, the zh_CN and zh_TW issue is not addressed in this release. Hopefully we can have a resolution in the next version.

helloworld1 avatar Jan 01 '23 04:01 helloworld1

well asap I have time I do have a look into it

My guess: we have to switch naming convention ... the question is if it will be supported out of the box

6543 avatar Jan 01 '23 19:01 6543

Thanks @6543 I can change all the locale manually to full local name. But it may break crowdin so I didn't do that.

helloworld1 avatar Jan 01 '23 23:01 helloworld1

#236

6543 avatar Jan 03 '23 00:01 6543

(I did not test if the android framework just pick that up or if it break things ... so that still has to be done)

6543 avatar Jan 03 '23 00:01 6543

Thanks a lot @6543. I will test out and let you know.

helloworld1 avatar Jan 03 '23 00:01 helloworld1

@6543 Unfortunately I reverted the PR https://github.com/helloworld1/FreeOTPPlus/pull/236. The locale resource name should be values-en-rUS or values-de-rDE. The PR is missing the "r" for the country name.

helloworld1 avatar Jan 03 '23 23:01 helloworld1

@helloworld1 are you sure! that's not spec compliant - what localization framework is used?

6543 avatar Jan 03 '23 23:01 6543

https://developer.android.com/guide/topics/resources/providing-resources#AlternativeResources Quote: The language is defined by a two-letter ISO 639-1 language code, optionally followed by a two letter ISO 3166-1-alpha-2 region code (preceded by lowercase r).

helloworld1 avatar Jan 03 '23 23:01 helloworld1

well there are standarts ... and there is android :/

I'll have a look

6543 avatar Jan 03 '23 23:01 6543

next try: #238 :laughing:

6543 avatar Jan 04 '23 04:01 6543

This time, it works great! Thanks!

helloworld1 avatar Jan 04 '23 06:01 helloworld1