social-app icon indicating copy to clipboard operation
social-app copied to clipboard

Japanese Language Update & Fix.

Open reindex-ot opened this issue 1 year ago • 13 comments

Update & Fix. Why don't you use Crowdin to manage the translation of this?

reindex-ot avatar Feb 22 '24 08:02 reindex-ot

There was a PR #2881 attempting to update the Japanese translation, perhaps you can help them review the translation. I agree with your perspective, we should use Crowdin or a similar platform to improve translations. This approach should also help reduce the workload for developers.

auroursa avatar Feb 22 '24 09:02 auroursa

I agree with your perspective, we should use Crowdin or a similar platform to improve translations. This approach should also help reduce the workload for developers.

@pfrazee commented here that it's on his TODOs to review this.

surfdude29 avatar Feb 22 '24 14:02 surfdude29

@tkusano @dolciss @oboenikui @noritada @middlingphys @hibiki https://github.com/bluesky-social/social-app/pull/2881 has been closed as new translation sections have been added in this PR. Please check again that the changes are correct, as this PR was not created by me.

Hima-Zinn avatar Feb 23 '24 02:02 Hima-Zinn

mainのほうで多くの変更があったようでconflictが発生してしまってます

tkusano avatar Feb 28 '24 12:02 tkusano

最新の物に合せて更新しました。 翻訳の追加分も翻訳済みです。

reindex-ot avatar Feb 29 '24 01:02 reindex-ot

ライト/ダーク/薄暗いについては、poファイル内の該当箇所にコメントで意図を書いておくのはどうでしょうか。 参考:The Format of PO Files Crowdin等であればそちらの機能が使えるのですが。 @Hima-Zinn

middlingphys avatar Feb 29 '24 06:02 middlingphys

mainをmergeした時 に、新規に追加された翻訳が巻き戻ってるようです

tkusano avatar Feb 29 '24 08:02 tkusano

タグやワードミュートに関する翻訳を戻しました。

reindex-ot avatar Feb 29 '24 09:02 reindex-ot

修正しました。ご確認ください。

reindex-ot avatar Mar 01 '24 02:03 reindex-ot

翻訳の修正はOKです。

ただ、mainのほうでmute関連のコードが修正され、msgid が "Mute all {tag} posts" のものが "Mute all {displayTag} posts" に変更になるなど、mainとmergeしてyarn intl:extractして確認しないといけないようです。

あと、招待コード関連のコードは大部分は消えたのですが、一部残ってるため、そこはfuzzyではなく、未訳状態になったようです。例: https://github.com/bluesky-social/social-app/blob/cf8b03801fd8e98bcd2da4d099a9dfbf5876de7d/src/view/com/auth/create/Step1.tsx#L170

tkusano avatar Mar 01 '24 06:03 tkusano

以上の修正はこちらの提案をもとにしてます https://bsky.app/profile/coffee-gkgk.bsky.social/post/3kmmhmpsuza2v

tkusano avatar Mar 01 '24 06:03 tkusano

ライト/ダーク/薄暗いについては、poファイル内の該当箇所にコメントで意図を書いておくのはどうでしょうか。

参考:The Format of PO Files

Crowdin等であればそちらの機能が使えるのですが。

@Hima-Zinn

意図というかなんか変に巻き戻されなくていればそのままだったんですけどね。

Hima-Zinn avatar Mar 01 '24 08:03 Hima-Zinn

I have reviewed all the changes, and it seems that the commentes on this PR are reflected.

tkusano avatar Mar 06 '24 01:03 tkusano

@tkusano @dolciss @oboenikui @noritada @middlingphys @hibiki @reindex-ot Due to the lack of progress in updating the Japanese-language display, it has been moved to https://github.com/bluesky-social/social-app/pull/3164. I apologize for the inconvenience, but please review again at the new location.

Hima-Zinn avatar Mar 10 '24 13:03 Hima-Zinn

@ maintainer of this repo. https://github.com/bluesky-social/social-app/pull/3164 is the new PR. Please note that this PR is old.

Hima-Zinn avatar Mar 10 '24 13:03 Hima-Zinn

NEW PR: https://github.com/bluesky-social/social-app/pull/3196

Hima-Zinn avatar Mar 13 '24 07:03 Hima-Zinn