Skip to content

Add Khmer Localization#6914

Merged
pfrazee merged 10 commits into
bluesky-social:mainfrom
roth-dev:KH-localization
Dec 18, 2024
Merged

Add Khmer Localization#6914
pfrazee merged 10 commits into
bluesky-social:mainfrom
roth-dev:KH-localization

Conversation

@roth-dev
Copy link
Copy Markdown
Contributor

@roth-dev roth-dev commented Dec 3, 2024

No description provided.

@roth-dev
Copy link
Copy Markdown
Contributor Author

roth-dev commented Dec 3, 2024

This will be close (#6897)

@surfdude29
Copy link
Copy Markdown
Contributor

It's always great to see another localization for Bluesky! 🦋🌐

Would it be possible to make three more changes in this PR to fully enable the Khmer localization please:

@surfdude29
Copy link
Copy Markdown
Contributor

Also, the devs ask for localisation PRs to be reviewed by at least one other native speaker; is there someone you could ask to do that? They don't necessarily have to have a GitHub account :)

@/pfrazee said this about the initial review on another localization PR:

To get this initial one merged, if another native speaker could just give it an initial review. We don't have to get really detailed, but we want to make sure it's close enough!

Comment thread src/locale/locales/km/messages.po Outdated

#: src/view/screens/Settings/index.tsx:455
#~ msgid "Accessibility settings"
#~ msgstr "ការកំណត់ភាពងាយស្រួល"
Copy link
Copy Markdown
Contributor

@surfdude29 surfdude29 Dec 3, 2024

Choose a reason for hiding this comment

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

Also, one tip for the future: the lines starting with #~ are obsolete strings that are no longer used in the app, so you can safely ignore them and there's no need to translate them. Some translators like to remove them to keep things tidy, but there's no need to do this if you'd prefer not to as they'll simply be ignored by the app.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Got it.

@roth-dev
Copy link
Copy Markdown
Contributor Author

roth-dev commented Dec 3, 2024

@surfdude29 Thank you for reviewing my PR. Now I have added the missing key.

Comment thread src/locale/locales/km/messages.po Outdated

#: src/Navigation.tsx:322
msgid "Accessibility Settings"
msgstr "ការកំណត់ភាពងាយស្រួល"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Accessibility Settings would be better if translated to ការកំណត់មានភាពងាយស្រួល

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A friendly tip in case it's helpful: you can suggest an alternate translation by clicking the ± button and then filling it in, e.g. like this:

Suggested change
msgstr "ការកំណត់ភាពងាយស្រួល"
msgstr "(suggested translation)"

If the PR author wants to accept the suggestion as it is, it's very easy for them to do so, and they can even accept many suggestions at once in a batch 💫

Comment thread src/locale/locales/km/messages.po Outdated

#: src/view/com/profile/ProfileMenu.tsx:108
msgid "Account muted"
msgstr "គណនីត្រូវបានបិទ"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Mute is different from Close and Block so I don't think the same translates as បិទ it's should be:
Mute = បិទសម្លេង
Block = បញ្ឈប់

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

So what would you suggest about Account muted in khmer?

@roth-dev
Copy link
Copy Markdown
Contributor Author

roth-dev commented Dec 3, 2024

@SokCheasrouch I really appreciate your time and effort in helping me out. Please confirm if you're done with review. And everything should be ready to go🚀

@SokCheasrouch
Copy link
Copy Markdown

@roth-dev I have just done my review

@roth-dev
Copy link
Copy Markdown
Contributor Author

roth-dev commented Dec 6, 2024

@surfdude29 All done

Comment thread src/locale/i18n.ts
@roth-dev
Copy link
Copy Markdown
Contributor Author

All updated according to sugesstion.

Co-authored-by: surfdude29 <149612116+surfdude29@users.noreply.github.com>
Copy link
Copy Markdown
Collaborator

@pfrazee pfrazee left a comment

Choose a reason for hiding this comment

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

Wow! Thanks so much!

@pfrazee pfrazee merged commit 323c974 into bluesky-social:main Dec 18, 2024
@roth-dev roth-dev mentioned this pull request Dec 19, 2024
Signez pushed a commit to Signez/bsky-social-app that referenced this pull request Dec 26, 2024
* add khmer language

* update translate

* update translate

* add missing localization key

* update khmer wording

* update khmer wording

* add missing translate

* add missing translate

* chore: add khmer local date-time format

* Update src/locale/i18n.ts

Co-authored-by: surfdude29 <149612116+surfdude29@users.noreply.github.com>

---------

Co-authored-by: Paul Frazee <pfrazee@gmail.com>
Co-authored-by: surfdude29 <149612116+surfdude29@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants