Skip to content

Conversation

jorgemmsilva
Copy link

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.

Copy link

cla-bot bot commented May 13, 2025

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
Copy link

CLAassistant commented May 13, 2025

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.
1 out of 2 committers have signed the CLA.

✅ jorgemmsilva
❌ calebguy
You have signed the CLA already but the status is still pending? Let us recheck it.

@jorgemmsilva
Copy link
Author

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

* example

* fix hardhat tests
Copy link

cla-bot bot commented Jun 11, 2025

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