metamask-mobile
metamask-mobile copied to clipboard
feat: add metametrics toast
Description
Related issues
Fixes: https://github.com/MetaMask/MetaMask-planning/issues/2437
Manual testing steps
- Go to this page...
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.
🔍 Existing Issues For Review
Your pull request is modifying functions with the following pre-existing issues:
📄 File: app/components/Nav/App/index.js
Function | Unhandled Issue |
---|---|
App |
Error: TaskQueue: Error with task : The 'navigation' object hasn't been initialized yet. This might happ... useEf... Event Count: 983 Affected Users: 1.1k |
Did you find this useful? React with a 👍 or 👎
Quality Gate passed
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
44.4% Coverage on New Code
0.0% Duplication on New Code