terminal-fi / swappa

Framework for executing cross-DEX token swaps
MIT License
20 stars 13 forks source link

UniswapV2 fixed fee need to be in bps so that it can represent a 25bps swap fee #36

Closed diwu1989 closed 2 years ago

diwu1989 commented 2 years ago

The current logic in UniswapV2 pair requires the fee to be defined & encoded as increments of 10bps Some UniswapV2 forks run with 25bps fee, like DXSwap on Gnosis

Need to re-write the pair + the offline logic to represent fees in bps instead of 10s of bps for flexibility.