-
Notifications
You must be signed in to change notification settings - Fork 419
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
LayerZero Hook and ISM #3094
LayerZero Hook and ISM #3094
Conversation
|
hey @ltyu just ran the checks for you - is this ready for review? excited to see all of your contributions recently |
Nope, this is about 40% done. Probably be done by next week since I will be on holiday. |
function _quoteDispatch( | ||
bytes calldata metadata, | ||
bytes calldata message | ||
) internal view virtual override returns (uint256) { | ||
bytes calldata lZMetadata = metadata.getCustomMetadata(); | ||
( | ||
uint16 dstChainId, | ||
address userApplication, | ||
, | ||
bytes memory payload, | ||
, | ||
bytes memory adapterParam | ||
) = parseLzMetadata(lZMetadata); | ||
|
||
(uint256 nativeFee, ) = lZEndpoint.estimateFees( | ||
dstChainId, | ||
userApplication, | ||
payload, | ||
false, | ||
adapterParam | ||
); | ||
return nativeFee; | ||
} |
Check warning
Code scanning / Slither
Unused return Medium
Incorrectly merging from main. Closing this for #3102. |
Description
Drive-by changes
Related issues
Backward compatibility
Yes
Testing
Manual/Unit Tests