-
Notifications
You must be signed in to change notification settings - Fork 65
Add OFTTransportAdapter
to support cross-chain token transfers of USDT0
via OFT
messaging protocol
#902
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
Add OFTTransportAdapter
to support cross-chain token transfers of USDT0
via OFT
messaging protocol
#902
Changes from all commits
088599a
d9bcdd0
cc464db
dc3da61
6898995
8d23a10
27452fb
68ea57e
abd7b98
9f18a3d
5402488
146f218
9f8ac2e
5fd6313
5aea206
6f27e94
d95be4c
1819983
0984f31
cbf9a0f
75b6577
086f4a9
6a0e0f4
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -16,6 +16,9 @@ import "@openzeppelin/contracts/token/ERC20/utils/SafeERC20.sol"; | |
contract Linea_SpokePool is SpokePool { | ||
using SafeERC20 for IERC20; | ||
|
||
// Linea_SpokePool does not use OFT messaging, setting the cap to 0 | ||
uint256 private constant OFT_FEE_CAP = 0; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It might in the future, right? We should probably pass this in the constructor There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The thing is, when we support this, we'll have to make changes to There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Another solution could be this: I add OFT code to every contract that "can potentially support OFT" and then in There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Cause currently only Arbitrum_SpokePool / Arbitrum_Adapter are using the OFT logic fully, and we'd need to change code for others to do the same. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Talked off-site about this. Will leave code like it is for now to not grow this PR any more |
||
|
||
/** | ||
* @notice Address of Linea's Canonical Message Service contract on L2. | ||
*/ | ||
|
@@ -51,7 +54,7 @@ contract Linea_SpokePool is SpokePool { | |
address _wrappedNativeTokenAddress, | ||
uint32 _depositQuoteTimeBuffer, | ||
uint32 _fillDeadlineBuffer | ||
) SpokePool(_wrappedNativeTokenAddress, _depositQuoteTimeBuffer, _fillDeadlineBuffer) {} // solhint-disable-line no-empty-blocks | ||
) SpokePool(_wrappedNativeTokenAddress, _depositQuoteTimeBuffer, _fillDeadlineBuffer, OFT_FEE_CAP) {} // solhint-disable-line no-empty-blocks | ||
|
||
/** | ||
* @notice Initialize Linea-specific SpokePool. | ||
|
Uh oh!
There was an error while loading. Please reload this page.