-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
fix: cp-7.56.1 dApp swaps failure on non-permitted network #20551
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
fix: cp-7.56.1 dApp swaps failure on non-permitted network #20551
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. |
Bitrise build: Fix works on iOS and Android physical devices. Alert is briefly shown when the first tx on a newly selected network is submitted, but tx submits succesfully. iOS: iOS-fix.mp4Android: android-fix.mov |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #20551 +/- ##
==========================================
+ Coverage 76.48% 76.50% +0.01%
==========================================
Files 3386 3388 +2
Lines 83430 83486 +56
Branches 15445 15454 +9
==========================================
+ Hits 63810 63869 +59
+ Misses 15325 15321 -4
- Partials 4295 4296 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Missing release label release-7.56.1 on PR. Adding release label release-7.56.1 on PR and removing other release labels(release-7.57.0), as PR was cherry-picked in branch 7.56.1. |
Description
Fixes balance loading issue when adding a new network from a dapp and immediately attempting to
transact. Previously, users would see no balance when trying to swap/transact on a newly added
non-permitted network, requiring them to navigate to the wallet home screen first to trigger balance
loading.
Changelog
CHANGELOG entry:Fixed issue where dapp swap transaction cannot complete on a non-permitted network
Related issues
Fixes: #19799
Manual testing steps
Screenshots/Recordings
~
Before
before.mp4
After
after.mp4
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Ensures AssetPollingProvider always polls the transaction’s chainId (deduped, EVM-only), and adds comprehensive tests for edge cases.
confirmation-asset-polling-provider.tsx
to buildpollChainIds
from EVM bridge chains and always include the transactionchainId
(asHex
), avoiding duplicates.transactionMeta?.chainId
.confirmation-asset-polling-provider.test.tsx
:chainId
even if not in bridge chains and no duplication when it is.Hex
and expectations for chainId handling.Written by Cursor Bugbot for commit 475f37c. This will update automatically on new commits. Configure here.