Skip to content

feat: add disableMessageFromOriginEvent to AbsInbox - #346

Open
jorgemmsilva wants to merge 2 commits into
OffchainLabs:mainfrom
SyndicateProtocol:disable-message-from-origin-event
Open

feat: add disableMessageFromOriginEvent to AbsInbox#346
jorgemmsilva wants to merge 2 commits into
OffchainLabs:mainfrom
SyndicateProtocol:disable-message-from-origin-event

Conversation

@jorgemmsilva

Copy link
Copy Markdown

The existing InboxMessageDeliveredFromOrigin event emitted by sendL2MessageFromOrigin omits the messageData to save gas. This forces off-chain indexers to retrieve the calldata for each tx, slowing down synchronization and increasing complexity.

This PR introduces a configurable option to change the event emitted bysendL2MessageFromOrigin in the Inbox contracts.
The goal is to enable faster and simpler synchronization for off-chain services, particularly for L3 appchains where minimizing L1/L2 settlement gas costs is less critical than data accessibility speed.

@cla-bot

cla-bot Bot commented May 13, 2025

Copy link
Copy Markdown

We require contributors to sign our Contributor License Agreement. In order for us to review and merge your code, please sign the linked documents below to get yourself added. https://na3.docusign.net/Member/PowerFormSigning.aspx?PowerFormId=b15c81cc-b5ea-42a6-9107-3992526f2898&env=na3&acct=6e152afc-6284-44af-a4c1-d8ef291db402&v=2

@CLAassistant

CLAassistant commented May 13, 2025

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 2 committers have signed the CLA.

❌ Jorge Silva
❌ calebguy


Jorge Silva seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@jorgemmsilva

Copy link
Copy Markdown
Author

pretty sure cla-bot is broken. I've signed that document multiple times.

* example

* fix hardhat tests
@cla-bot

cla-bot Bot commented Jun 11, 2025

Copy link
Copy Markdown

We require contributors to sign our Contributor License Agreement. In order for us to review and merge your code, please sign the linked documents below to get yourself added. https://na3.docusign.net/Member/PowerFormSigning.aspx?PowerFormId=b15c81cc-b5ea-42a6-9107-3992526f2898&env=na3&acct=6e152afc-6284-44af-a4c1-d8ef291db402&v=2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants