-
Notifications
You must be signed in to change notification settings - Fork 10
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
feat: Configurable chain in quickstart #153
Conversation
28d8905
to
81fe59a
Compare
043c6de
to
2b1e513
Compare
- Tested for execution on Base - all good.
2b1e513
to
9b1e03e
Compare
- Need this for the latest executor addresses
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.
Thank you so much for figuring this out @tamaralipows 🙏🏼
- Excluding Balancer V2 since it's not supported on tycho indexer for base - Also get rid of the base hardcoding when creating txs and provider. Get this from the cli input
Tested that all still works properly.
5d5fb3f
to
9d29ab6
Compare
We support multiple chains so this may not just be ETH, but perhaps a Base URL for example. It now matched tycho-execution.
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.
Thank you so much @tamaralipows 🙏🏼 only have tiny tiny requests!
This PR is included in version 0.80.0 🎉 |
No description provided.