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

chore: translations for ticket fields module #551

Merged
merged 1 commit into from
Dec 9, 2024

Conversation

gosiexon-zen
Copy link
Contributor

Description

This PR adds the source file for the translations used in the Ticket Fields module. We are using ticket fields in two modules now so we decided to separate fields from new request form. I am targeting the master branch because our internal tooling is fetching the translations from master. So, to start the translation process, we need to push the YAML file on master even if the code is still in the feature branch: https://github.com/zendesk/copenhagen_theme/tree/service-catalog-eap

Screenshots

Checklist

  • 📗 all commit messages follow the conventional commits standard
  • ⬅️ changes are compatible with RTL direction
  • ♿ Changes to the UI are tested for accessibility and compliant with WCAG 2.1.
  • 📝 changes are tested in Chrome, Firefox, Safari and Edge
  • 📱 changes are responsive and tested in mobile
  • 👍 PR is approved by @zendesk/vikings

@gosiexon-zen gosiexon-zen requested a review from a team as a code owner December 5, 2024 11:22
@gosiexon-zen
Copy link
Contributor Author

Hi @kubraokcu we moved translations for ticket fields from new request form module to separate module so the keys were changes. This was also checked already in PR's to Service Catalog feature branch but please have a second look 🙏

key: "cph-theme-ticket-fields.credit-card-digits-hint"
title: "Additional label shown above the credit card field, informing the user that only the last 4 digits are required."
screenshot: "https://drive.google.com/file/d/13Pz2p5q7FHX9FKT6MUR6Sg5vWy55X0kP/view?usp=drive_link"
value: "(Last 4 digits)"

Choose a reason for hiding this comment

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

Didn't we have a variable for the credit card field name?
{field_name} (last 4 digits)
Here the space between the name and the suffix will be hard-coded, it's not localization-friendly for Asian and Cyrillic languages which have different space width or no space at all

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We decided to do this that way in other PR months ago here was the conversation: #474

Choose a reason for hiding this comment

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

Ahh thank you, apologies I didn't remember about this conversation

@kubraokcu kubraokcu added the g11n-commented Commented by Globalization label label Dec 5, 2024
key: "cph-theme-ticket-fields.credit-card-digits-hint"
title: "Additional label shown above the credit card field, informing the user that only the last 4 digits are required."
screenshot: "https://drive.google.com/file/d/13Pz2p5q7FHX9FKT6MUR6Sg5vWy55X0kP/view?usp=drive_link"
value: "(Last 4 digits)"

Choose a reason for hiding this comment

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

Ahh thank you, apologies I didn't remember about this conversation

@kubraokcu kubraokcu added g11n-approved Commented by Globalization label and removed g11n-commented Commented by Globalization label labels Dec 6, 2024
@gosiexon-zen gosiexon-zen merged commit 31a4463 into master Dec 9, 2024
5 checks passed
@gosiexon-zen gosiexon-zen deleted the mbien/ticket-fields-translations branch December 9, 2024 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
g11n-approved Commented by Globalization label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants