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

evm: Add support for adapter instructions #47

Merged
merged 5 commits into from
Dec 12, 2024

Conversation

bruce-riley
Copy link
Collaborator

@bruce-riley bruce-riley commented Dec 9, 2024

This PR modifies sendMessage and quoteDeliveryPrice to take encoded adapter instructions. It parses them into the per-adapter instructions and passes them into the adapter functions.

To do this, it adds a new AdapterInstructions library along with tests for it.

@bruce-riley bruce-riley force-pushed the evm_adapter_instructions branch from 8fd6445 to f442f4f Compare December 9, 2024 19:14
@bruce-riley bruce-riley force-pushed the evm_adapter_instructions branch 2 times, most recently from 51b59d0 to eefdf75 Compare December 10, 2024 14:30
@bruce-riley bruce-riley force-pushed the evm_adapter_instructions branch from eefdf75 to 82533d5 Compare December 10, 2024 15:16
@bruce-riley bruce-riley force-pushed the evm_adapter_instructions branch from 45b047e to 8f9b270 Compare December 10, 2024 16:34
@bruce-riley bruce-riley changed the title evm: Make getSendAdaptersByChain also return index evm: Add support for adapter instructions Dec 10, 2024
@bruce-riley bruce-riley marked this pull request as ready for review December 10, 2024 17:59
.gitmodules Show resolved Hide resolved
panoel
panoel previously approved these changes Dec 11, 2024
Copy link
Collaborator

@panoel panoel left a comment

Choose a reason for hiding this comment

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

Thanks for fixing my spelling errors.

@bruce-riley
Copy link
Collaborator Author

Created issue #48 to investigate whether we should pass the instructions in as an array rather than as encoded bytes. For now, will stick with encoded bytes because that is what NTT is doing.

@bruce-riley bruce-riley merged commit 5fffb3e into main Dec 12, 2024
5 checks passed
@bruce-riley bruce-riley deleted the evm_adapter_instructions branch December 12, 2024 22:57
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.

2 participants