fix: release-blocker restore gas estimate alerts cp-7.78.0#30266
fix: release-blocker restore gas estimate alerts cp-7.78.0#30266maxime-oe wants to merge 1 commit into
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. |
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 08c4b6f. Configure here.
08c4b6f to
bc78256
Compare
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection: Performance Test Selection: |
|




Description
Fixes release blocker regression #30260 for release 7.78.0
Changelog
CHANGELOG entry: restores gas estimate alerts on all networks except strictly gas-sponsored ones
Related issues
Fixes: #30260
Manual testing steps
Same testing intructions as for #29835
Add non-regression for issues raised in #30260
Screenshots/Recordings
Before
After
Pre-merge author checklist
Performance checks (if applicable)
trace()for usage andaddTokenfor an exampleFor performance guidelines and tooling, see the Performance Guide.
Pre-merge reviewer checklist
Note
Medium Risk
Changes the conditions for showing the gas estimate failed warning based on a remote feature-flagged sponsored-network map and hardware-wallet detection, which could affect when users see (or miss) an important fee warning.
Overview
Restores the
GasEstimateFailedwarning logic to be suppressed only when the currentchainIdis flagged as gas-fee-sponsored, instead of relying onuseIsGaslessSupported/pending checks.The hook now reads
transactionMetato determinechainIdandfromaddress, and it explicitly keeps the alert enabled for hardware-wallet accounts even on sponsored networks. Tests were updated to mock the sponsored-network selector andisHardwareAccount, and to cover the sponsored-network and hardware-wallet exception cases.Reviewed by Cursor Bugbot for commit bc78256. Bugbot is set up for automated code reviews on this repo. Configure here.