Skip to content
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

chore: Remove bridge binding from BatchPoster #2715

Merged

Conversation

epociask
Copy link
Contributor

@epociask epociask commented Oct 1, 2024

Noticed that a delayed inbox binding was being generated and persisted into the batch poster struct. Couldn't find any references of this being used outside of the bridgeAddr for generating sequencer inbox submission tx access lists.

@cla-bot cla-bot bot added the s Automatically added by the CLA bot if the creator of a PR is registered as having signed the CLA. label Oct 1, 2024
@tsahee tsahee self-requested a review October 12, 2024 01:02
Copy link
Contributor

@tsahee tsahee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tsahee tsahee enabled auto-merge October 14, 2024 23:42
@tsahee tsahee merged commit 65d7401 into OffchainLabs:master Oct 15, 2024
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design-approved s Automatically added by the CLA bot if the creator of a PR is registered as having signed the CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants