Conversation
|
|
Current dependencies on/for this PR:
This comment was auto-generated by Graphite. |
|
This PR changes implementation code, but doesn't include a changeset. Did you forget to add one? |
|
Is the plan to test this using the There are currently linting errors, will review again once fixed |
|
Correct, the last PR in the chain will have op-e2e tests. I’ll fix the linter. |
8470ebe to
197ceb1
Compare
I'm realizing there isn't a good way to include legacy contracts in |
197ceb1 to
1d0516c
Compare
|
Hey @mslipper! This PR has merge conflicts. Please fix them before continuing review. |
bb3c79b to
5127b51
Compare
I don't think we need to - they're already in bindings, and I can use a real network to test legacy support for the indexer. |
5127b51 to
d2180a2
Compare

Updates the indexer's DB package to support both Bedrock and the legacy system. 90% of of this is a revert back to Mark's last commit on the indexer pre-Bedrock.