Skip to content

Add Greek Language ('el') Support and Internationalization#6677

Merged
pfrazee merged 11 commits into
bluesky-social:mainfrom
imertz:main
Dec 31, 2024
Merged

Add Greek Language ('el') Support and Internationalization#6677
pfrazee merged 11 commits into
bluesky-social:mainfrom
imertz:main

Conversation

@imertz
Copy link
Copy Markdown
Contributor

@imertz imertz commented Nov 23, 2024

This PR adds full internationalization support for the Greek language, using the ISO 639-1 language code 'el'.

@pfrazee
Copy link
Copy Markdown
Collaborator

pfrazee commented Nov 23, 2024

Amazing. Are there any native greek speakers available to review this?

@ipinak
Copy link
Copy Markdown

ipinak commented Nov 24, 2024

@pfrazee I can help with that. Should we ensure all things are translated on this PR or can it be split in smaller PRs?

@auroursa
Copy link
Copy Markdown
Contributor

@pfrazee I can help with that. Should we ensure all things are translated on this PR or can it be split in smaller PRs?

I think it's beneficial to have a good start for a first translation PR, so it would be best to continue improving this PR until it's ready to be merged.

Copy link
Copy Markdown

@bekatoros bekatoros left a comment

Choose a reason for hiding this comment

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

Inline comments for changes or additions.

Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po
Comment thread src/locale/locales/el/messages.po Outdated
Copy link
Copy Markdown

@ipinak ipinak left a comment

Choose a reason for hiding this comment

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

Here are some comments to begin with. Let me know what you think about them.

Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po
Comment thread src/locale/locales/el/messages.po
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
@pfrazee
Copy link
Copy Markdown
Collaborator

pfrazee commented Nov 27, 2024

Everyone here is amazing -- for the initial translation and the review. I'm going to let this keep cooking because it looks like not everything is resolved yet.

Comment thread src/locale/locales/el/messages.po Outdated
Copy link
Copy Markdown

@dpkgme dpkgme left a comment

Choose a reason for hiding this comment

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

Έκανα πάλι ένα πέρασμα.
Προτείνω όπου starter pack => πακέτο έναρξης.
Για τα feeds μάλλον τείνω προς ροές.
Κάτι που με προβληματίζει λίγο είναι η συμπεριληπτικότητα σε κάποια σημεία, π.χ. έχω προτείνει "Θέλετε σίγουρα...;αντί γιαΕίστε σίγουροι...;`. Ας το σκεφτούμε.

Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po Outdated
Comment thread src/locale/locales/el/messages.po
Comment thread src/locale/locales/el/messages.po Outdated
@surfdude29
Copy link
Copy Markdown
Contributor

surfdude29 commented Dec 2, 2024

Would it be possible to add one more change to this PR to enable the Greek localization? If you could add an 'el', entry to app.config.js here please, that would be great:

social-app/app.config.js

Lines 101 to 102 in 770eeb5

'de',
'es',

This will enable Greek to be displayed as a supported language in the App Store listing, and it will also allow the TextInput context menus to be translated into Greek for iOS users who have this set as their device language :)

@imertz
Copy link
Copy Markdown
Contributor Author

imertz commented Dec 12, 2024

Would it be possible to add one more change to this PR to enable the Greek localization? If you could add an 'el', entry to app.config.js here please, that would be great:

social-app/app.config.js

Lines 101 to 102 in 770eeb5

'de',
'es',

This will enable Greek to be displayed as a supported language in the App Store listing, and it will also allow the TextInput context menus to be translated into Greek for iOS users who have this set as their device language :)

Added!

@imertz
Copy link
Copy Markdown
Contributor Author

imertz commented Dec 12, 2024

@ipinak @dpkgme @bekatoros @stergioc
Thank you so much for your comments and help.
Some notes:
I believe it's best to go for "Starter Pack" and not "πακέτο έναρξης", as the greek users are accustomed (I believe) to the meaning of Starter Pack and also this is the way it is used in their conversations in BlueSky.
Also:
feed: ροή
like: Μου αρέσει

I don't know if I am forgetting something.

I am doing a last pass of the messages.po, to see if it's missing something and I will push the commit.

@surfdude29
Copy link
Copy Markdown
Contributor

Would it be possible to add one more change to this PR to enable the Greek localization? If you could add an 'el', entry to app.config.js here please, that would be great:

social-app/app.config.js

Lines 101 to 102 in 770eeb5

'de',
'es',

This will enable Greek to be displayed as a supported language in the App Store listing, and it will also allow the TextInput context menus to be translated into Greek for iOS users who have this set as their device language :)

Added!

I'm not seeing app.config.js here, maybe there was a problem pushing the commit? 🤔

@ipinak
Copy link
Copy Markdown

ipinak commented Dec 13, 2024

@ipinak @dpkgme @bekatoros @stergioc Thank you so much for your comments and help. Some notes: I believe it's best to go for "Starter Pack" and not "πακέτο έναρξης", as the greek users are accustomed (I believe) to the meaning of Starter Pack and also this is the way it is used in their conversations in BlueSky. Also: feed: ροή like: Μου αρέσει

I don't know if I am forgetting something.

I am doing a last pass of the messages.po, to see if it's missing something and I will push the commit.

@imertz I agree with what you said.

@imertz
Copy link
Copy Markdown
Contributor Author

imertz commented Dec 13, 2024

Would it be possible to add one more change to this PR to enable the Greek localization? If you could add an 'el', entry to app.config.js here please, that would be great:

social-app/app.config.js

Lines 101 to 102 in 770eeb5

'de',
'es',

This will enable Greek to be displayed as a supported language in the App Store listing, and it will also allow the TextInput context menus to be translated into Greek for iOS users who have this set as their device language :)

Added!

I'm not seeing app.config.js here, maybe there was a problem pushing the commit? 🤔

I did a last check on the messages.po.
Commited and pushed!

Comment thread src/locale/i18n.ts
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.

Exciting!! Thanks so much everybody!

Comment thread src/locale/i18n.ts Outdated
@surfdude29
Copy link
Copy Markdown
Contributor

@pfrazee I saw that you're merging some translation PRs, so maybe this one could be too? :)

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.

Agree! Awesome, thanks all!

@pfrazee pfrazee merged commit 2300410 into bluesky-social:main Dec 31, 2024
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.

9 participants