Skip to content
This repository has been archived by the owner on Mar 6, 2024. It is now read-only.

string-import-02-25-2024 #4055

Merged
merged 2 commits into from
Feb 28, 2024
Merged

string-import-02-25-2024 #4055

merged 2 commits into from
Feb 28, 2024

Conversation

github-actions[bot]
Copy link
Contributor

This (automated) PR import string changes

@isabelrios
Copy link
Collaborator

@Delphine tagging you for review because I'm not sure about those changes, if they are correct or not..

@Delphine
Copy link
Contributor

Delphine commented Feb 26, 2024

@isabelrios Hmm I don't see anything concerning looking at this. But is there something in particular that you've noticed and you think could be an error in this string import PR?

@@ -44,7 +44,7 @@
"actionSheet.openIn" = "Відкрити в %@";

/* Button to dismiss the 'Add Pass Failed' alert. */
"AddPass.Error.Dismiss" = "Гаразд";
"AddPass.Error.Dismiss" = "OK";
Copy link
Collaborator

Choose a reason for hiding this comment

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

@Delphine I was surprised about this change which seems like undoing a translated string...

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh I see! Some languages actually decide to use "OK" in some cases. I've checked across other strings and seems like this is expected for Ukrainian. Thanks for checking!

Copy link
Collaborator

Choose a reason for hiding this comment

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

Sounds good to me then! Thanks!

@mergify mergify bot merged commit 2413d36 into main Feb 28, 2024
8 checks passed
@mergify mergify bot deleted the string-import-02-25-2024 branch February 28, 2024 08:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants