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

CCIP Adapter #26

Draft
wants to merge 31 commits into
base: master
Choose a base branch
from
Draft

CCIP Adapter #26

wants to merge 31 commits into from

Conversation

jiro-ono
Copy link
Contributor

@jiro-ono jiro-ono commented Nov 4, 2023

PR-Codex overview

Focus of this PR:

This PR focuses on adding a CCIPAdapter to the project and implementing the necessary functions for swapping and bridging tokens using the CCIP protocol.

Detailed summary:

  • Added CCIPAdapter.sol which implements the ISushiXSwapV2Adapter interface and the CCIPReceiver contract.
  • Implemented the swap and executePayload functions in the CCIPAdapter contract.
  • Added the getFee function to calculate the fees in native tokens for the bridge/message.
  • Added the RpSentNativeIn, NotCCIPRouter, InsufficientGas, and NotEnoughNativeForFees error messages.
  • Added the CCIPBridgeParams struct to store the parameters for bridging tokens using CCIP.
  • Updated the CCIPAdapterSwapAndBridgeTest test contract to test the functionality of the CCIPAdapter contract.

The following files were skipped due to too many changes: src/adapters/CCIPAdapter.sol, test/CCIPAdapterTests/CCIPAdapterBridgeTest.t.sol, test/CCIPAdapterTests/CCIPAdapterExecutesTest.t.sol

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

ccip-develop
@jiro-ono jiro-ono marked this pull request as draft November 4, 2023 02:51
@jiro-ono
Copy link
Contributor Author

jiro-ono commented Nov 4, 2023

CL team is gonna re-deploy the router / contracts when they make this available for general release.

Will comeback to this then, and we'll need to change out the addresses used for testing to verify everything looks good.

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.

1 participant