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

[strings] vault corruption recovery strings #5465

Merged
merged 2 commits into from
Jan 11, 2023
Merged

Conversation

owencraston
Copy link
Contributor

@owencraston owencraston commented Jan 10, 2023

Development & PR Process

  1. Follow MetaMask Mobile Coding Standards
  2. Add release-xx label to identify the PR slated for a upcoming release (will be used in release discussion)
  3. Add needs-dev-review label when work is completed
  4. Add needs-qa label when dev review is completed
  5. Add QA Passed label when QA has signed off

Description

Write a short description of the changes included in this pull request, also include relevant motivation and context. Have in mind the following questions,
1. What is the reason for the change?

  • get the strings merged so we can get them translated before we merge the recovery flow
    2. What is the improvement/solution?
  • based on the designs here

Screenshots/Recordings

Simulator Screen Shot - iPhone 11 Pro - 2023-01-10 at 15 23 43
Simulator Screen Shot - iPhone 11 Pro - 2023-01-10 at 15 26 19

Issue

Progresses #???

Checklist

  • There is a related GitHub issue
  • Tests are included if applicable
  • Any added code is fully documented

@owencraston owencraston requested a review from a team as a code owner January 10, 2023 00:36
@github-actions
Copy link
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@owencraston owencraston added No QA Needed Apply this label when your PR does not need any QA effort. Code Impact - Low Minor code change that can safely applied to the codebase team-accounts needs-dev-review PR needs reviews from other engineers (in order to receive required approvals) labels Jan 10, 2023
@coreyjanssen
Copy link
Contributor

Looks great! My only comment is to add "wallet" to the "Try recovering" CTA on the New wallet needed screen. So it will appear as Try recovering wallet.

locales/languages/en.json Outdated Show resolved Hide resolved
locales/languages/en.json Outdated Show resolved Hide resolved
locales/languages/en.json Outdated Show resolved Hide resolved
@owencraston owencraston force-pushed the vault-corruption-strings branch 2 times, most recently from c3636b2 to a9fc147 Compare January 10, 2023 23:37
@coreyjanssen
Copy link
Contributor

good to go from a content design perspective!

locales/languages/en.json Outdated Show resolved Hide resolved
@owencraston owencraston force-pushed the vault-corruption-strings branch from a9fc147 to 83551ea Compare January 11, 2023 00:20
Copy link
Contributor

@Cal-L Cal-L left a comment

Choose a reason for hiding this comment

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

LGTM

@owencraston owencraston merged commit 3d92b24 into main Jan 11, 2023
@owencraston owencraston deleted the vault-corruption-strings branch January 11, 2023 00:50
@github-actions github-actions bot locked and limited conversation to collaborators Jan 11, 2023
@gantunesr gantunesr removed the needs-dev-review PR needs reviews from other engineers (in order to receive required approvals) label Jan 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Code Impact - Low Minor code change that can safely applied to the codebase No QA Needed Apply this label when your PR does not need any QA effort. team-accounts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants