-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
feat: add GeneralAlertBanner
component
#13627
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
Conversation
…tamask-mobile into feat/add-alert-modal
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. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## feat/add-alert-confirm #13627 +/- ##
=========================================================
Coverage ? 62.07%
=========================================================
Files ? 1995
Lines ? 44140
Branches ? 5977
=========================================================
Hits ? 27401
Misses ? 14926
Partials ? 1813 ☔ View full report in Codecov by Sentry. |
4bc10cb
to
c3a2e9e
Compare
|
|
|
Description
PR adding
GeneralAlertBanner
component which is part of the Alert System.Until we add an alert there is no functional change in the app.
Related issues
Fixes: https://github.com/MetaMask/mobile-planning/issues/2140
Manual testing steps
Screenshots/Recordings
content-message.mp4
info-warning.mp4
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist