-
Notifications
You must be signed in to change notification settings - Fork 638
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
Swaps: logic, data flow, and performance fixes #5787
Merged
Merged
Changes from 26 commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
9031a8e
Add pause functionality to useAnimatedTime
christianbaroni 1cf77b9
useSyncSharedValue: allow pauseSync to be a boolean
christianbaroni d568c25
Create useDelayedMount hook
christianbaroni 283dba4
Fix swap status bar handling
christianbaroni 936a3e4
Refactor FastFallbackCoinIconImage to use faster-image
christianbaroni d80d2a7
Fix animated coin icon jank
christianbaroni 1f45265
Lots of logic, performance, UI fixes
christianbaroni cbe04fd
Remove commented code
christianbaroni f400dd7
Merge branch 'develop' into @christian/swaps-fixes
christianbaroni b5d51c1
Fix shared value dependency
christianbaroni df75557
Fix initialInputNativeValue formatting
christianbaroni 5d6fbad
lint: uncomment store favorites
christianbaroni 6c15099
Update default swaps chain order
christianbaroni 4a66612
Prevent NaN input values
christianbaroni 4983197
Memoize UserAssetsSync
christianbaroni 833fe3b
cleanup routes
walmat 354d59a
fix double all networks on android
walmat dd90c84
Merge branch 'develop' into @christian/swaps-fixes
walmat bf8abe0
fix APP-1535
walmat 6ba95bb
Merge branch '@christian/swaps-fixes' of https://github.com/rainbow-m…
walmat 4624fd9
"Fetching Prices" → "Fetching"
christianbaroni e70dc6b
Delete userAssetsStore favorites migration
christianbaroni 9f11757
Fix remaining logic flaws, userAssetsStore improvements, misc. fixes
christianbaroni 51cdcdc
Consolidate useSwapEstimatedGasFee state updates
christianbaroni 78882b1
Clean up EstimatedSwapGasFee styles
christianbaroni 6577e90
Remove redundant memoization
christianbaroni a9ea6bf
Fix output asset balance and badge
christianbaroni 484b638
Clear output asset when wallet address changes if no user assets
christianbaroni 39d82d2
Use userAssets store for tokenToSellList data; fast search; fix searc…
christianbaroni 27363e5
Add animated coin icon border radius back into animatedIconSource on …
christianbaroni 5140f5d
Remove log
christianbaroni 7eb8126
Fix coin row token balance
christianbaroni b57ff0b
Fix lint error — add toScaledIntegerWorklet
christianbaroni b270fbd
Merge branch 'develop' into @christian/swaps-fixes
christianbaroni e9b372a
Merge branch 'develop' into @christian/swaps-fixes
christianbaroni fd16c4b
Use token decimals for balance comparison
christianbaroni 0b0491f
Fix toScaledIntegerWorklet scaling, add tests
christianbaroni cc7a642
Persist associatedWalletAddress, chainBalances, bump store version
christianbaroni 07b47b7
Fix userAssetsQuery errors
christianbaroni File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removing this caused https://linear.app/rainbow/issue/APP-1534/some-images-displayed-as-rectangles-on-android