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

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from
Open

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 a9108de):

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

(expires Wed, 29 Jan 2025 13:28:19 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
"coupon_not_valid": "Gutscheincode ist ungültig",
"coupon_not_valid_for_offer": "Gutscheincode ist für dieses Angebot nicht gültig",
"credit_card": "Kreditkarte",
"credit_card_name": "Kreditkartename",

Choose a reason for hiding this comment

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

Suggested change
"credit_card_name": "Kreditkartename",
"credit_card_name": "Kreditkartenname",

"card_lock": "Artikel gesperrt",
"close_modal": "Panel schließen",
"confirmation_dialog": {
"close": "Abbrechen",

Choose a reason for hiding this comment

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

There are som einconsistencies between formal and informal tone (Du vs Sie) - any preferences here?

Personally, would opt for informal.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks @morrieinmaas! I think using informal tone is fine, wdyt @AntonLantukh? For Dutch, we do use a formal tone, which is more standard (although perhaps not modern and "hip")

Choose a reason for hiding this comment

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

No problem and my pleasure! I'll update the lot of it in the near future. You canm decide which you prefer. Had a brief chat with @AntonLantukh on slack - apparently Netflix and spiegel use the formal one. So i think formal it is.

Will update in one I got some time spare.

@morrieinmaas
Copy link

@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

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

@AntonLantukh
Copy link
Collaborator

@morrieinmaas thanks a lot!

"copied_url": "URL copiée",
"current_episode": "Épisode actuel",
"current_video": "Vidéo actuelle",
"episode": "épisode",
Copy link

Choose a reason for hiding this comment

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

Should the first letter be capitalized?

"geo_blocked_description": "Nous nous excusons, mais ce contenu est actuellement indisponible dans votre région.",
"geo_blocked_title": "Contenu indisponible"
},
"season": "saison",
Copy link

Choose a reason for hiding this comment

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

Should the first letter be capitalized?

@@ -0,0 +1,19 @@
{
"error_description": "Rechargez cette page ou réessayez plus tard.",
Copy link

Choose a reason for hiding this comment

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

Rafraîchissez cette page ou réessayez plus tard.

"submit_config_id": "Soumettre",
"use_the_jwp_dashboard": "Vous n'avez pas de configuration ? Utilisez le tableau de bord JWP pour créer une configuration d'application ou cliquez sur Annuler pour afficher une démo générique.",
"view_generic_demo": "Afficher une démo générique ?",
"viewing_config_demo": "Cela vous redirigera vers une application démo générique. Êtes-vous sûr de vouloir continuer ?"
Copy link

Choose a reason for hiding this comment

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

Cela vous redirigera vers une application de démo générique. Êtes-vous sûr de vouloir continuer ?

"co": "Colorado",
"ct": "Connecticut",
"de": "Delaware",
"dc": "District of Columbia",
Copy link

Choose a reason for hiding this comment

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

Le District de Columbia

"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?

"kr": "Corée, République de",
"kw": "Koweït",
"kg": "Kirghizistan",
"la": "République démocratique populaire lao",
Copy link

Choose a reason for hiding this comment

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

République démocratique populaire du Laos

"ni": "Nicaragua",
"ne": "Niger",
"ng": "Nigéria",
"nu": "Niue",
Copy link

Choose a reason for hiding this comment

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

Niué

"ws": "Samoa",
"sm": "Saint-Marin",
"st": "Sao Tomé-et-Principe",
"sa": "Arabie saoudite",
Copy link

Choose a reason for hiding this comment

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

Arabie Saoudite

"sz": "Eswatini",
"se": "Suède",
"ch": "Suisse",
"sy": "République arabe syrienne",
Copy link

Choose a reason for hiding this comment

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

Syrie

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