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

IOS-2592 wording conflicts card #234

Merged
merged 14 commits into from
Dec 9, 2022
Merged

Conversation

megakoko
Copy link
Contributor

@megakoko megakoko commented Nov 24, 2022

первый подход к синхронизации

common_warning -> dialog_attestation_completed_with_warnings_title
pin_change_code_format -> pin_set_code_format
enter_user_code_button_title_forgot -> reset_codes_btn_forgot_your_code
error_not_supported_firmware_version -> error_old_firmware
attestation_failed_card -> dialog_attestation_did_failed_message
attestation_online_failed_title -> dialog_attestation_completed_offline_title
attestation_online_failed_body -> dialog_attestation_completed_offline_message
@megakoko megakoko marked this pull request as ready for review November 24, 2022 13:58
Copy link
Collaborator

@tureck1y tureck1y left a comment

Choose a reason for hiding this comment

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

Мне не нравятся ключи dialog_. У нас нет диалогов и ломается система именования ключей. Что-то с префикса начинается, что-то нет

@@ -7,17 +7,17 @@
"common_continue" = "Fortsetzen";
"common_success" = "Erfolg";
"common_error" = "Fehler";
"common_warning" = "Warning";
"dialog_attestation_completed_with_warnings_title" = "Warning";
Copy link
Collaborator

Choose a reason for hiding this comment

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

ключ какой-то неудачный

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Этот ключ в одном месте используется, поэтому не заморачивался и взял из андроида

"attestation_online_failed_title" = "Online attestation failed";
"attestation_online_failed_body" = "We cannot finish card's online attestation at this time. You can continue at your own risk and try again later, retry now or cancel the operation";
"attestation_warning_attest_wallets" = "Too many runs of Attest Wallet or Sign looks suspicious.";
"dialog_attestation_did_failed_message" = "This card may be production sample or counterfeit. You can continue at your own risk";
Copy link
Collaborator

Choose a reason for hiding this comment

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

зря затащили андроидовские слова в ключах к нам, лучше бы конечно более универсальные ключи использовать, если не сложно, лучше сразу поменять

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Переименовал обратно

@megakoko megakoko merged commit 0d2c527 into develop Dec 9, 2022
@megakoko megakoko deleted the IOS-2592-wording-conflicts-card branch December 9, 2022 09:11
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