Skip to content

Commit

Permalink
feat: adds the new default settings view to onboarding (#24562)
Browse files Browse the repository at this point in the history
fixes: MetaMask/MetaMask-planning#2080

--


<br class="Apple-interchange-newline">

## **Description**

Introduces the new Default Settings view and Congratulations views
depending on the fact if user imported a wallet or created a new one.

**User imported a wallet:**
![Screenshot 2024-06-03 at 16 08
44](https://github.com/MetaMask/metamask-extension/assets/1125631/c9784cbb-e4e2-4557-b6f6-527d9df91fa5)

**User created a new wallet and backed up the seed phrase:**
![Screenshot 2024-06-03 at 16 11
13](https://github.com/MetaMask/metamask-extension/assets/1125631/8045e007-63bb-4aac-915c-d908d03b52a1)

**User created a new wallet and didn’t back up the seed phrase:**
![Screenshot 2024-06-03 at 16 09
37](https://github.com/MetaMask/metamask-extension/assets/1125631/005d34ec-587a-4978-92d3-0ed14d64f9c8)

**Inside look on the Default Settings:**
![Screenshot 2024-06-03 at 16 13
49](https://github.com/MetaMask/metamask-extension/assets/1125631/c573c7ba-36f4-4b46-8124-9c2091018356)
![Screenshot 2024-06-03 at 16 14
00](https://github.com/MetaMask/metamask-extension/assets/1125631/733ab5db-6aec-43a9-8da8-f1d543e5d3a1)


## **Related issues**

Fixes:

## **Manual testing steps**

1. Go to this page...
2.
3.

## **Screenshots/Recordings**

<!-- If applicable, add screenshots and/or recordings to visualize the
before and after of your change. -->

### **Before**

<!-- [screenshots/recordings] -->

### **After**

<!-- [screenshots/recordings] -->

## **Pre-merge author checklist**

- [ ] I’ve followed [MetaMask Coding
Standards](https://github.com/MetaMask/metamask-extension/blob/develop/.github/guidelines/CODING_GUIDELINES.md).
- [ ] I've completed the PR template to the best of my ability
- [ ] I’ve included tests if applicable
- [ ] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [ ] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-extension/blob/develop/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.

## **Pre-merge reviewer checklist**

- [ ] I've manually tested the PR (e.g. pull and build branch, run the
app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described
in the ticket it closes and includes the necessary testing evidence such
as recordings and or screenshots.

---------

Co-authored-by: vinnyhoward <[email protected]>
Co-authored-by: David Walsh <[email protected]>
  • Loading branch information
3 people authored Oct 10, 2024
1 parent 0798717 commit 61adc78
Show file tree
Hide file tree
Showing 35 changed files with 1,174 additions and 1,287 deletions.
42 changes: 0 additions & 42 deletions app/_locales/de/messages.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

42 changes: 0 additions & 42 deletions app/_locales/el/messages.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 61adc78

Please sign in to comment.