-
Notifications
You must be signed in to change notification settings - Fork 0
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: proxy for KatanaV3Factory and NonfungiblePositionManager #7
Conversation
…CODE_POINTER can be verified on sourcify
treasury = _treasury; | ||
emit TreasuryChanged(address(0), _treasury); | ||
function initialize(address owner_, address treasury_) external { | ||
require(!_initialized); |
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.
provide error message
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.
please storage gap for all logics for proxy contracts.
emit OwnerChanged(address(0), _owner); | ||
constructor() { | ||
// disable initialization | ||
_initialized = true; |
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.
emit event
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.
can we not used Initializable extension?
function initializeImmutables(address factory_, address token0_, address token1_, uint24 fee_, int24 tickSpacing_) | ||
public | ||
override | ||
{ | ||
require(!_immutablesInitialized); |
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.
provide error message
@@ -125,8 +125,10 @@ contract KatanaV3Pool is IKatanaV3Pool { | |||
_immutablesInitialized = true; |
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.
emit event
emit OwnerChanged(address(0), _owner); | ||
constructor() { | ||
// disable initialization | ||
_initialized = true; |
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.
can we not used Initializable extension?
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.
please add gap and disable initializer
…proxy chore(`proxy`): merge from `release/v1.0.0`
Description
Checklist