-
Notifications
You must be signed in to change notification settings - Fork 202
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
refactor: Remove useArbTokenBridge from store #1479
Closed
chrstph-dvx
wants to merge
49
commits into
master
from
1432-use-usenetworksrelationship-inside-usearbtokenbridge
Closed
Changes from 3 commits
Commits
Show all changes
49 commits
Select commit
Hold shift + click to select a range
1a053aa
Remove useArbTokenBridge from store
chrstph-dvx 0e08dfd
Update syncers
chrstph-dvx 14f7d20
Add useCallback to useArbTokenBridge, move addBridgeTokenListToBridge
chrstph-dvx 955547e
Memoize useTransactions
chrstph-dvx 2d39062
Fix typing
chrstph-dvx 4ad3bb5
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
chrstph-dvx 7dc7d8c
Reset useArbTokenBridge
chrstph-dvx 4cf85e2
Add shared zustand store for bridgeTokens
chrstph-dvx c02f0e5
Replace params with useNetworks in useArbTokenBridge
chrstph-dvx a96c5a6
Add addBridgeTokenListToBridge to useArbTokenBridge
chrstph-dvx d2e4846
Move updateTokenData
chrstph-dvx 1ffeabb
Move addToken
chrstph-dvx 65583ee
Move addToExecutedMessagesCache
chrstph-dvx b87dc6f
Add useCallback for updateEthBalances
chrstph-dvx b4da51b
Add useCallback for useArbTokenBridge functions
chrstph-dvx 207c6f9
Memoize returned object in useArbTokenBridge
chrstph-dvx 8cef344
Remove unused import
chrstph-dvx aa1b575
Rework call to useArbTokenBridge
chrstph-dvx f2c7c82
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
chrstph-dvx 59f56e4
Keep track of source/destination chain in state
chrstph-dvx 5354099
Fix warning with missing autoresizer style
chrstph-dvx b6503cd
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
chrstph-dvx 130e603
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
chrstph-dvx 5576101
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
chrstph-dvx dcb5145
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
chrstph-dvx f96c6a7
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
chrstph-dvx 22b6448
Remove ledger from yarn.lock
chrstph-dvx 96e9124
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
chrstph-dvx 0af6601
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
chrstph-dvx 8be9478
fix import dialog
fionnachan cfcadf2
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
fionnachan 5df85c3
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
chrstph-dvx 801d145
Fix import token
chrstph-dvx 38cf72a
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
chrstph-dvx d8e12da
Add missing hooks in App
chrstph-dvx 3bd8a45
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
fionnachan 303d33d
Remove connectionState
chrstph-dvx ce1ade6
Remove unused import
chrstph-dvx b868e0e
Add lost code after merge
chrstph-dvx a99a910
Remove commented code
chrstph-dvx fe8b01a
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
fionnachan 7dd835d
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
chrstph-dvx 24440ca
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
chrstph-dvx d915ff1
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
chrstph-dvx e52eebe
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
fionnachan 396576e
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
chrstph-dvx 04c22f8
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
chrstph-dvx d7a8593
Merge branch 'master' into 1432-use-usenetworksrelationship-inside-us…
chrstph-dvx 30462e6
Remove extra token button
chrstph-dvx 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
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The diff on this file is actually fairly simple:
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It can be browsed from commit 7dc7d8c |
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.
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.
useAppState
is now an unused import