-
Notifications
You must be signed in to change notification settings - Fork 203
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 2/3 #2089
base: fs-1001/add-useBridgeTokensStore
Are you sure you want to change the base?
refactor: Remove useArbTokenBridge from store 2/3 #2089
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
c72b3a9
to
5789015
Compare
cb9121a
to
747c8f4
Compare
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.
other than that one small comment, everything else looks good
@@ -417,130 +451,157 @@ export const useArbTokenBridge = ( | |||
] | |||
) |
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.
there's a missing dependency here, i know you didn't change it but can you add destinationAddress
to the dep array please?
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.
Fixed in a376a1a
(#2089)
b09b794
to
31b866b
Compare
5789015
to
94da1e5
Compare
31b866b
to
54e32b1
Compare
a376a1a
to
718b487
Compare
Summary
Wrap methods with useCallbacks in useArbTokenBridge
Steps to test