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
NFT metadata: allow token0 numerator for certain pair ratios #60
Conversation
|
||
function initialize(TokenRatioOrderPriority[] memory tokens) public initializer() { | ||
for (uint256 i = 0; i < tokens.length; i++) { | ||
tokenRatioPriority[tokens[i].token] = tokens[i].priority; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe should trigger an event for each token to have a record of which tokens were recorded...(helpful for making changes in the future)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, just gotta find out why the gas snapshots changed
@@ -39,6 +55,7 @@ contract NonfungibleTokenPositionDescriptor is INonfungibleTokenPositionDescript | |||
token1Symbol: SafeERC20Namer.tokenSymbol(token1), | |||
token0Decimals: IERC20Metadata(token0).decimals(), | |||
token1Decimals: IERC20Metadata(token1).decimals(), | |||
hasToken0RatioNumerator: hasToken0RatioNumerator(token0, token1), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would it make more sense to name this something like flipRatio?
|
||
exports[`SwapRouter gas tests #exactInput WETH9 -> 0 1`] = `111324`; | ||
exports[`SwapRouter gas tests #exactInput WETH9 -> 0 1`] = `111179`; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why did this happen?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mm..not sure..this file changed a bit in my last PR too. Happened on merge
No description provided.