metamask-extension
metamask-extension copied to clipboard
feat: New onboarding text
Description
Updates the "Help us improve MetaMask" copy during onboarding.
Related issues
Fixes: https://github.com/MetaMask/MetaMask-planning/issues/2336
Manual testing steps
- Start onboarding
- Check that the copy in the 'Help us improve MetaMask' screen matches the new copy.
- Test the Privacy Policy link
Screenshots/Recordings
Before
After
Pre-merge author checklist
- [ ] I’ve followed MetaMask Coding Standards.
- [ ] 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 format if applicable
- [ ] I’ve applied the right labels on the PR (see labeling guidelines). 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.
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.
Except for test-unit-jest-main
(which should become green after the above-suggested changes), other failures seem unrelated to this PR
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 67.43%. Comparing base (
d27a233
) to head (941467f
). Report is 59 commits behind head on develop.
Additional details and impacted files
@@ Coverage Diff @@
## develop #24050 +/- ##
===========================================
+ Coverage 67.37% 67.43% +0.06%
===========================================
Files 1278 1289 +11
Lines 49881 50249 +368
Branches 12944 13027 +83
===========================================
+ Hits 33605 33883 +278
- Misses 16276 16366 +90
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Builds ready [a7cb32d]
- builds: chrome, firefox
- builds (beta): chrome
- builds (flask): chrome, firefox
- builds (MMI): chrome, firefox
- builds (test): chrome, firefox
- builds (test-flask): chrome, firefox
- build viz: Build System
- mv3: Background Module Init Stats
- mv3: UI Init Stats
- mv3: Module Load Stats
- mv3: Bundle Size Stats
- mv2: E2e Actions Stats
- code coverage: Report
- storybook: Storybook
- typescript migration: Dashboard
- all artifacts
Page Load Metrics (452 ± 441 ms)
Platform | Page | Metric | Min (ms) | Max (ms) | Average (ms) | StandardDeviation (ms) | MarginOfError (ms) |
---|---|---|---|---|---|---|---|
Chrome | Home | firstPaint | 66 | 168 | 87 | 26 | 12 |
domContentLoaded | 9 | 51 | 16 | 12 | 6 | ||
load | 54 | 2765 | 452 | 918 | 441 | ||
domInteractive | 9 | 51 | 16 | 12 | 6 |
Bundle size diffs [🚨 Warning! Bundle size has increased!]
- background: 0 Bytes (0.00%)
- ui: 3.18 KiB (0.05%)
- common: 1.51 KiB (0.02%)
Builds ready [adcdba8]
- builds: chrome, firefox
- builds (beta): chrome
- builds (flask): chrome, firefox
- builds (MMI): chrome, firefox
- builds (test): chrome, firefox
- builds (test-flask): chrome, firefox
- build viz: Build System
- mv3: Background Module Init Stats
- mv3: UI Init Stats
- mv3: Module Load Stats
- mv3: Bundle Size Stats
- mv2: E2e Actions Stats
- code coverage: Report
- storybook: Storybook
- typescript migration: Dashboard
- all artifacts
Page Load Metrics (1544 ± 787 ms)
Platform | Page | Metric | Min (ms) | Max (ms) | Average (ms) | StandardDeviation (ms) | MarginOfError (ms) |
---|---|---|---|---|---|---|---|
Chrome | Home | firstPaint | 73 | 202 | 117 | 44 | 21 |
domContentLoaded | 10 | 44 | 17 | 8 | 4 | ||
load | 59 | 3879 | 1544 | 1639 | 787 | ||
domInteractive | 10 | 44 | 17 | 8 | 4 |
Bundle size diffs [🚨 Warning! Bundle size has increased!]
- background: 0 Bytes (0.00%)
- ui: 2.81 KiB (0.04%)
- common: 1.51 KiB (0.02%)