fix: wallet issues OK-50640 OK-50547 OK-50483 OK-50482 OK-50436 OK-50411#10408
Open
weatherstar wants to merge 14 commits intoxfrom
Open
fix: wallet issues OK-50640 OK-50547 OK-50483 OK-50482 OK-50436 OK-50411#10408weatherstar wants to merge 14 commits intoxfrom
weatherstar wants to merge 14 commits intoxfrom
Conversation
…Networks loading stuck Use a stable React tree structure (Stack with dynamic flexDirection) instead of conditional XStack/YStack branches so that TokenListBlock is never unmounted/remounted when the viewport crosses the mobile/desktop breakpoint.
Add checkShouldRetryBroadcastTx override for Algorand vault to handle "cannot broadcast txns in follower mode" errors with exponential backoff.
…licate txid The 3-minute memoized cache on _getSuggestedParams caused identical firstRound/lastRound values across consecutive transactions, producing the same txid and triggering "transaction already in ledger" errors.
Reset accountWorth and DeFi overview when wallet ID changes, not just for allNetworks or unbackuped wallets. Add accountId mismatch guard in balanceString useMemo to avoid displaying previous wallet balance during the render-effect timing gap.
keepLocalCopy returns percent-encoded file URIs, causing RNFS to fail when filenames contain Chinese or other non-ASCII characters.
Use CollapsibleTabContext to programmatically scroll the collapsible tab header when the keyboard covers the input, and replace the plain ScrollView with KeyboardAwareScrollView for the not-backed-up wallet case.
weatherstar
requested review from
ezailWang,
huhuanming,
originalix,
revan-zhang and
sidmorizon
as code owners
Contributor
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.