feat: gas sponsorship improve alerts and remove gas alert#42300
Conversation
|
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. |
✨ Files requiring CODEOWNER review ✨✅ @MetaMask/confirmations (4 files, +123 -28)
|
41fa35b to
aa3ad19
Compare
aa3ad19 to
952d8e1
Compare
952d8e1 to
c234e5a
Compare
c234e5a to
62ed846
Compare
62ed846 to
61a49e4
Compare
Builds ready [61a49e4]
⚡ Performance Benchmarks (Total: 🟢 0 pass · 🟡 0 warn · 🔴 0 fail)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
61a49e4 to
4421c30
Compare
Builds ready [d903558]
⚡ Performance Benchmarks (Total: 🟢 0 pass · 🟡 0 warn · 🔴 0 fail)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit e331749. Configure here.
|
Builds ready [afbc28c]
⚡ Performance Benchmarks (Total: 🟢 15 pass · 🟡 10 warn · 🔴 0 fail)
Bundle size diffs
|




Description
This PR contains two fixes related to Gas Sponsorship alerts:
Changelog
CHANGELOG entry: remove gas alerts from confirmation modal in gasless flows
CHANGELOG entry: update copy of 10 MON minimal reserve confirmation alert
Related issues
Fixes: https://consensyssoftware.atlassian.net/browse/NEB-951?atlOrigin=eyJpIjoiMDBmNDlhY2E5ZDZiNGQ0NGIwYzNkZGEwMWQ4Y2QxZjYiLCJwIjoiaiJ9
Manual testing steps
Test 1, send MON beyond
10 MONreserve:On Monad (gas sponsored, but 10 MON reserve limit), try to send:
MON.MON- 1.MON- 10.MON-9.999Observe that the 10 MON error message is always displayed first, as opposed as before, and that user cannot proceed with the transaction.
Test 2, removal of network fee warnings:
Screenshots/Recordings
Before
Failing tx on SEI:

Send MON beyond 10 MON reserve:

After
Failing tx on SEI:

Send MON beyond 10 MON reserve:

Pre-merge author checklist
Pre-merge reviewer checklist
Note
Medium Risk
Changes confirmation alert gating and severity for gasless/sponsored transactions; incorrect conditions could hide or block critical user warnings during transaction confirmation.
Overview
Avoids showing the
gasEstimateFailednetwork-fee alert when the gasless support check is pending or when gasless is supported, preventing irrelevant “update gas limit” prompts in gasless/sponsored flows.Reworks the Monad gas sponsorship reserve warning into a blocking
Severity.Dangeralert (gasSponsorshipAlert) with a Buy native token action and updated, reserve-focused copy/reason, and updates related i18n strings (addsalertMinimumReserve, removesalertReasonGasSponsorshipUnavailable, and tweaks reserve wording).Reviewed by Cursor Bugbot for commit afbc28c. Bugbot is set up for automated code reviews on this repo. Configure here.