clabby
clabby
Current dependencies on/for this PR: * develop * **PR #4807** 👈 This comment was auto-generated by [Graphite](https://app.graphite.dev/github/pr/ethereum-optimism/optimism/4807?utm_source=stack-comment).
@mergifyio requeue
Current dependencies on/for this PR: * develop * **PR #4954** 👈 This comment was auto-generated by [Graphite](https://app.graphite.dev/github/pr/ethereum-optimism/optimism/4954?utm_source=stack-comment).
> Just wanted to record here the alternative idea of only finalizing the withdrawal if the external call succeeds. > > It feels a bit off because there is also...
Closing in favor of #5017
This LGTM, nice idea. We need to re-enable the `rethdb-reader` tests in CI, will approve once we've re-enabled and rebased on `develop` 👍
> @clabby Hi! I see that this PR is going to be closed, can it be merged?🥹 Apologies for the delay here! Once #9692 is merged, we can rebase this...
h/t @sebastianst, I think we should take a slightly different direction here. Right now, even though the `op-node` only needs 1 DB, the Go API allows for creating multiple `RethDBReceiptsFetcher`s....
Current dependencies on/for this PR: * **#9591** 👈 * `develop` This [stack of pull requests](https://stacking.dev/?utm_source=stack-comment) is managed by [Graphite](https://app.graphite.dev/github/pr/ethereum-optimism/optimism/9591?utm_source=stack-comment).
> What's the status of this PR? Are we still using this approach for safety mode? Nah, don't think so. Will move into draft for reference in case the other...