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

Update translations: merchant #3404

Closed

Conversation

translation-platform[bot]
Copy link
Contributor

This PR modifies translations in the project. It will append multiple commits and may add, modify, or delete translations. Feel free to merge any time.

Before you deploy

  • Check translations for malicious HTML.

Merge conflicts?

If this PR has been open long enough that it has drifted out of sync from the base branch and it has developed merge conflicts as a result, follow these steps, in order:

  1. Close this PR.
  2. Run the following Spy command in Slack, inserting the PR number in the second argument:

@spy translation-platform force-pull Shopify/dawn pull_request_number=<this PR>.

Translation Platform will recreate the translation commits based on the latest commit on the base branch and open a new PR.

Questions?

Visit #help-localization for questions about this pull request.

Translation component owners: @Shopify/themes-translations

@translation-platform
Copy link
Contributor Author

🕐 This PR has been open for 2 days now 🕐

It is best to merge it as soon as possible in order to avoid having to deal with merge conflicts.

More importantly, the content here really wants to reach production so everyone can benefit from it.

It is your responsibility to merge this PR once you've reviewed it. 🙏

💚

Copy link
Contributor

@ludoboludo ludoboludo left a comment

Choose a reason for hiding this comment

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

Same concern here of going back to an anglo term instead of keeping the language specific term.

@translation-platform
Copy link
Contributor Author

🕐 This PR has been open for 4 days now 🕐

It is best to merge it as soon as possible in order to avoid having to deal with merge conflicts.

More importantly, the content here really wants to reach production so everyone can benefit from it.

It is your responsibility to merge this PR once you've reviewed it. 🙏

💚

@translation-platform
Copy link
Contributor Author

🕐 This PR has been open for 7 days now 🕐

It is best to merge it as soon as possible in order to avoid having to deal with merge conflicts.

More importantly, the content here really wants to reach production so everyone can benefit from it.

It is your responsibility to merge this PR once you've reviewed it. 🙏

💚

@@ -610,7 +610,7 @@
"label": "Muotokuva"
},
"options__3": {
"label": "Neliö"
"label": "Square"
Copy link
Contributor

Choose a reason for hiding this comment

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

It looks like we are still getting this issue 🤔 @ludoboludo I see your comment above. Do you have context on next steps?

Copy link
Contributor

Choose a reason for hiding this comment

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

I had shared the issue with the localization team and they were looking into it. I followed up with them today to mention that it's still an issue 👍

@@ -610,7 +610,7 @@
"label": "Muotokuva"
},
"options__3": {
"label": "Neliö"
"label": "Square"
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this is still not supposed to be that way. Shouldn't be the english word for it.

@@ -1659,7 +1659,7 @@
"label": "Cercle"
},
"options__2": {
"label": "Carré"
"label": "Square"
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here, should be Carré

@@ -2515,7 +2515,7 @@
"label": "Cercle"
},
"options__2": {
"label": "Carré"
"label": "Square"
Copy link
Contributor

Choose a reason for hiding this comment

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

Here as well. It's not consistent through that file. Sometimes it's good sometimes it isn't/

@@ -728,7 +728,7 @@
"label": "Retrato"
},
"options__3": {
"label": "Quadrado"
"label": "Square"
Copy link
Contributor

Choose a reason for hiding this comment

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

Should probably not be square in this file either.

@@ -610,7 +610,7 @@
"label": "縱向"
},
"options__3": {
"label": "正方形"
"label": "Square"
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here

@translation-platform
Copy link
Contributor Author

🕐 This PR has been open for 9 days now 🕐

It is best to merge it as soon as possible in order to avoid having to deal with merge conflicts.

More importantly, the content here really wants to reach production so everyone can benefit from it.

It is your responsibility to merge this PR once you've reviewed it. 🙏

💚

@translation-platform
Copy link
Contributor Author

🕐 This PR has been open for 11 days now 🕐

It is best to merge it as soon as possible in order to avoid having to deal with merge conflicts.

More importantly, the content here really wants to reach production so everyone can benefit from it.

It is your responsibility to merge this PR once you've reviewed it. 🙏

💚

@ludoboludo
Copy link
Contributor

Closing PR for now as per conversation with localization team.

@ludoboludo ludoboludo closed this Apr 17, 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.

2 participants