Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat / Add new languages (nl/fr/de) #651

Merged
merged 7 commits into from
Feb 28, 2025

Conversation

langemike
Copy link
Collaborator

This change adds and activates 3 new languages:

  • Dutch
  • French
  • German

I kept us_state.json in English.

These are all translated using ChatGTP.

Tickets: OTT-2756 & OTT-2763

Copy link

github-actions bot commented Nov 22, 2024

Visit the preview URL for this PR (updated for commit a8b4255):

https://ottwebapp--pr651-feat-new-languages-d2tl7s32.web.app

(expires Sat, 29 Mar 2025 10:37:02 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c198f8a3a199ba8747819f7f1e45cf602b777529

Copy link
Collaborator

@royschut royschut left a comment

Choose a reason for hiding this comment

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

Nice!

Base automatically changed from feat/missing-translations to develop November 25, 2024 11:30
@morrieinmaas
Copy link
Contributor

@ChristiaanScheermeijer @AntonLantukh - changed everything to formal tone in latest commit. I'll have a re-read at some time to make sure everything is gucci. Meanwhile have a great couple of christmas days

@morrieinmaas
Copy link
Contributor

Adjusted to the best of my abilities - @AntonLantukh @ChristiaanScheermeijer

@AntonLantukh
Copy link
Collaborator

@morrieinmaas thanks a lot!

"pending_downgrade": "Diminution en attente",
"pending_offer_switch": "Sera mis à jour vers \"{{title}}\" après la prochaine date de facturation",
"price_paid_with": "{{price}} payé avec {{method}}",
"price_paid_with_card": "Prix payé avec carte",
Copy link

Choose a reason for hiding this comment

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

Prix payé par carte, but we should use avec for cash payments. What are the potential methods in the line above?

Copy link
Collaborator

Choose a reason for hiding this comment

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

price_paid_with is used for all other payment methods which can be PayPal or In-app purchase. The price_paid_with_card is exclusively used for creditcard payments.

@ChristiaanScheermeijer
Copy link
Collaborator

@qlecorre thank you very much for your help with this PR! 💪

I have processed most of your suggestions and commented on your questions. My goal is to merge this PR this week. Let me know if you are unavailable this week, then I will proceed with merging this PR and create a new issue for the remaining translation discussions.

@ChristiaanScheermeijer ChristiaanScheermeijer merged commit edec3e5 into develop Feb 28, 2025
10 checks passed
@ChristiaanScheermeijer ChristiaanScheermeijer deleted the feat/new-languages branch February 28, 2025 09:18
@ChristiaanScheermeijer
Copy link
Collaborator

I have created #673 for the remaining feedback

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.

6 participants