-
Notifications
You must be signed in to change notification settings - Fork 41
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
Conversation
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
…wording-conflicts-card
…pleted_with_warnings_message
There was a problem hiding this 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"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ключ какой-то неудачный
There was a problem hiding this comment.
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"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
зря затащили андроидовские слова в ключах к нам, лучше бы конечно более универсальные ключи использовать, если не сложно, лучше сразу поменять
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Переименовал обратно
…line_failed_title
…online_failed_body
…ation_warning_attest_wallets
первый подход к синхронизации