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

Fixes translation issue of dynamic variables in workflows #4734

Merged
merged 4 commits into from Sep 29, 2022

Conversation

CarinaWolli
Copy link
Member

What does this PR do?

Variables with translations that have more than 2 words like nom de l'événement were falsely translated to
{NOM_DE L'ÉVÉNEMENT} instead of {NOM_DE_L'ÉVÉNEMENT}

With this PR all spaces get replaced with underscores and not just the first one.

Environment: Staging(main branch) / Production

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How should this be tested?

  • Create a workflow with a custom template
  • Add variables to the custom template
  • Go to settings and choose French as your language
  • Go back to your workflow and see that the variables are translated correctly
  • Change language back to English
  • Go back to your workflow
  • See that variables are in English again

Checklist

  • I haven't added tests that prove my fix is effective or that my feature works

@CarinaWolli CarinaWolli requested a review from a team September 28, 2022 13:22
@vercel
Copy link

vercel bot commented Sep 28, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cal ✅ Ready (Inspect) Visit Preview Sep 29, 2022 at 0:56AM (UTC)

Copy link
Contributor

@emrysal emrysal left a comment

Choose a reason for hiding this comment

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

Leaving the final approval to @zomars

@@ -255,7 +255,7 @@
"user_needs_to_confirm_or_reject_booking_recurring": "{{user}} muss noch die Terminreihe des wiederkehrenden Termins bestätigen oder ablehnen.",
"meeting_is_scheduled": "Dieser Termin ist geplant",
"meeting_is_scheduled_recurring": "Ihr wiederkehrender Termin wurde geplant",
"submitted": "Ihr Termin wurde gebuhct",
"submitted": "Ihr Termin wurde gebucht",
Copy link
Contributor

Choose a reason for hiding this comment

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

Good spot!

Copy link
Member

@zomars zomars left a comment

Choose a reason for hiding this comment

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

Ship it

@zomars zomars merged commit 47df213 into main Sep 29, 2022
@zomars zomars deleted the fix/workflow-variable-language branch September 29, 2022 13:07
@PeerRich PeerRich added the core area: core, team members only label Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core area: core, team members only
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants