-
Notifications
You must be signed in to change notification settings - Fork 429
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
Support msg.value in native hooks #2410
Comments
This was referenced Aug 7, 2023
aroralanuk
added a commit
that referenced
this issue
Aug 15, 2023
### Description - Updated the OP Stack tests for the Mailbox V3 transient storage version - Allowing OPStackHook to send msg.value at the time of message delivery (uses bit masking) - Added LibBit library, will be useful for all the auth hooks which can send `msg.value` ### Drive-by changes - None ### Related issues - Fixes breaking OP Stack tests for hyperlane-xyz/issues#513 - Also fixes #2410 ### Backward compatibility No ### Testing Unit tests --------- Co-authored-by: Yorke Rhodes <[email protected]>
yorhodes
added a commit
that referenced
this issue
Aug 18, 2023
### Description - Updated the OP Stack tests for the Mailbox V3 transient storage version - Allowing OPStackHook to send msg.value at the time of message delivery (uses bit masking) - Added LibBit library, will be useful for all the auth hooks which can send `msg.value` ### Drive-by changes - None ### Related issues - Fixes breaking OP Stack tests for hyperlane-xyz/issues#513 - Also fixes #2410 ### Backward compatibility No ### Testing Unit tests --------- Co-authored-by: Yorke Rhodes <[email protected]>
yorhodes
added a commit
that referenced
this issue
Sep 13, 2023
- Updated the OP Stack tests for the Mailbox V3 transient storage version - Allowing OPStackHook to send msg.value at the time of message delivery (uses bit masking) - Added LibBit library, will be useful for all the auth hooks which can send `msg.value` - None - Fixes breaking OP Stack tests for hyperlane-xyz/issues#513 - Also fixes #2410 No Unit tests --------- Co-authored-by: Yorke Rhodes <[email protected]> Remove e2e and add yarn build to CI Adding protocol fees (#2640) - Adding protocol fee as a hook - None V3 Yes Fuzz tests --------- Co-authored-by: Yorke Rhodes <[email protected]>
OPStack supports msg.value now |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The text was updated successfully, but these errors were encountered: