ccip icon indicating copy to clipboard operation
ccip copied to clipboard

MultiOnRamp fee token configs

Open RayXpub opened this issue 1 year ago • 2 comments

Motivation

Implements per destination chain fee token configs in the EVM2EVMMultiOnRamp, which allows the OnRamp to support more than one destination chain configuration.

Solution

Removes the FeeTokenConfig struct by:

  • Outsourcing destination chain specific fields to the DestChainConfig struct, namely networkFeeUSDCents & gasMultiplierWeiPerEth which are now shared across all fee tokens
  • Renaming premiumMultiplierWeiPerEth to feeTokenConfig which now becomes per fee token vs per destination chain per fee token
  • Replacing the isEnabled flag by feeTokenConfig > 0

Out of scope for this PR:

  • Full coverage
  • Token transfer fee configs to be per destination chain

RayXpub avatar May 15 '24 14:05 RayXpub

Go solidity wrappers are out-of-date, regenerate them via the make wrappers-all command

github-actions[bot] avatar May 15 '24 14:05 github-actions[bot]

LCOV of commit f2f482b during Solidity Foundry #4795

Summary coverage rate:
  lines......: 98.7% (1262 of 1279 lines)
  functions..: 96.4% (268 of 278 functions)
  branches...: 92.2% (535 of 580 branches)

Files changed coverage rate: n/a

github-actions[bot] avatar May 15 '24 14:05 github-actions[bot]