-
Notifications
You must be signed in to change notification settings - Fork 10
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
feat: Add new pool initialization action and validation at PM call #26
Conversation
CI fail as expected due to exceed bytecode size. see #27 on the passing PR |
src/modules/V3ToV4Migrator.sol
Outdated
error InvalidAction(bytes4 action); | ||
error OnlyMintAllowed(); | ||
error OnlyAddLiqudityAllowed(); | ||
error BlacklistedAction(); |
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.
This is duplicate with OnlyMintAllowed/OnlyAddLiqudityAllowed?
… feat/add-action-protect-pm-v2
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
[Part 2 of 2] feat: Add new pool initialization action and validation at PM call
uint256 numActions = actions.length; | ||
|
||
for (uint256 actionIndex = 0; actionIndex < numActions; actionIndex++) { | ||
uint256 action = uint8(actions[actionIndex]); |
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 casting uint8 here for a uint256 variable?
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.
actions[actionIndex]
returns a bytes1 and we need to explictly convert it to a uint
type
Update comment. |
This PR does the following: