-
Notifications
You must be signed in to change notification settings - Fork 483
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 subgraph to support multiple chains #233
refactor subgraph to support multiple chains #233
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
42e6436
to
75fb116
Compare
75fb116
to
c1f81e9
Compare
1ecc521
to
655d764
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.
Huge! Will let @ianlapham dig in more.
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.
Looks good! just some nits
only question is, would changing the schema be a big lift for our apps? wondering if its finally time to change derivedETH
to derivedNative
if too much work seems fine
@ianlapham i think it would require a resync so it wouldn't be a lot of active work but a lot of passive waiting and coming back to the subgraphs when they finish syncing. i don't think it's worth it to do now but if we ever need to resync the subgraphs for whatever reason that would be a good opportunity to do so |
b658543
to
04c87b1
Compare
04c87b1
to
3ab5b27
Compare
Refactors
main
to support deploying all networks of the v3 subgraph