-
Notifications
You must be signed in to change notification settings - Fork 5.4k
feat: shield subscription error state toasts #36718
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
- banner radius to 12px - button radius 8px and medium text
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. |
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [a5dae8b]
UI Startup Metrics (1256 ± 69 ms)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [6b061bf]
UI Startup Metrics (1223 ± 58 ms)
Bundle size diffs [🚀 Bundle size reduced!]
|
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [00568d5]
UI Startup Metrics (1224 ± 66 ms)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [bd18b04]
UI Startup Metrics (1238 ± 72 ms)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [d4bd7fb]
UI Startup Metrics (1224 ± 64 ms)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
- updated the shield library
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [541d9fd]
UI Startup Metrics (1248 ± 68 ms)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Description
Added toasts to display Shield subscription's error states.
Shield payment declined
Shield coverage ending
Changelog
CHANGELOG entry: Added shield subscription error toasts
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Adds Shield “payment declined” and “coverage ending” toasts, with persisted dismiss state, selectors, and subscription paused/ending logic, plus i18n and tests.
ShieldPausedToast
andShieldEndingToast
intoast-master.js
; wire intoToastMaster
and navigate toTRANSACTION_SHIELD_ROUTE
on action.selectShowShieldPausedToast
andselectShowShieldEndingToast
.setShieldPausedToastLastClickedOrClosed
andsetShieldEndingToastLastClickedOrClosed
.shieldPausedToastLastClickedOrClosed
andshieldEndingToastLastClickedOrClosed
(defaults, metadata, setters) inapp-state-controller.ts
, exposed viametamask-controller.js
.shared/lib/shield.ts
):getShieldSubscription
; addgetIsShieldSubscriptionPaused
andgetIsShieldSubscriptionEndingSoon
; keepgetIsShieldSubscriptionActive
.PausedSubscriptionStatuses
inshared/constants/subscriptions.ts
.transaction-shield.tsx
.app/_locales/en*.json
.Written by Cursor Bugbot for commit 541d9fd. This will update automatically on new commits. Configure here.