-
Notifications
You must be signed in to change notification settings - Fork 8
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
Swap query support for hooks that require account #404
base: main
Are you sure you want to change the base?
Conversation
{ blockNumber: block }, | ||
); | ||
const { result } = account | ||
? // pseudo query fails if account has insufficient allowance or balance |
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.
Don't have a big preference on ternary vs another method but did find it pretty tricky to notice the ?
with the comment inline like this.
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.
ya that is pretty bad 😬
I couldn't quite remember what you wanted me to do with the caveat. Should it be a comment in code or is it just something to keep in mind moving forward?
Closes #398
Status: On Hold
Summary
account
toquerySingleSwap
andqueryBatchSwap
( for v3 only )pnpm example ./examples/swaps/customSwap.ts
account
is my dev wallet which holds the mock veBAL the hooks contract checks for