-
Notifications
You must be signed in to change notification settings - Fork 42
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
[REG-1430] RegistrarCustody contract #368
base: main
Are you sure you want to change the base?
Conversation
0c9418e
to
ac62b75
Compare
ac62b75
to
457aece
Compare
_disableInitializers(); | ||
} | ||
|
||
function initialize(address _unsRegistry, address _mintingManager) public initializer { |
Check warning
Code scanning / Slither
Conformance to Solidity naming conventions Warning
_disableInitializers(); | ||
} | ||
|
||
function initialize(address _unsRegistry, address _mintingManager) public initializer { |
Check warning
Code scanning / Slither
Conformance to Solidity naming conventions Warning
function _msgData() internal view override(ContextUpgradeable, ERC2771RegistryContext) returns (bytes calldata) { | ||
return ERC2771RegistryContext._msgData(); | ||
} |
Check warning
Code scanning / Slither
Dead-code Warning
457aece
to
821990b
Compare
821990b
to
6b6a567
Compare
Contracts size report
|
function safeTransfer(address to, uint256 tokenId) external onlyMinter nonReentrant { | ||
_protectTokenOperation(tokenId); | ||
|
||
unsRegistry.setOwner(to, tokenId); | ||
|
||
delete virtualOwners[tokenId]; | ||
} |
Check notice
Code scanning / Slither
Reentrancy vulnerabilities Low
External calls:
- unsRegistry.setOwner(to,tokenId)
State variables written after the call(s):
- delete virtualOwners[tokenId]
PR Checklist
1. Contracts versioning
patch
version of the contracts is increased if changes have been made to theUNSRegistry
,MintingManager
,ProxyReader
,ENSCustody
contracts.minor
version of the contracts is increased if breaking changes have been made to theUNSRegistry
,MintingManager
,ProxyReader
,ENSCustody
contracts. It includes changes of interfaces.2. Contracts licensing
3. Coverage
4. Configs versioning
uns-config.json
is increased if changes have been made to the config.ens-config.json
is increased if changes have been made to the config.resolver-keys.json
is increased if changes have been made to the config.ens-resolver-keys.json
is increased if changes have been made to the config.5. Package versioning
patch
version of package is increased if valuable changes have been made to the package. It includes contracts update, configs update, etc.major.minor
version of package is synced with version ofUNSRegistry
contract.CHANGELOG
is updated with short description for the new version.6. Code review
resolver-keys.json
code review is required from DevTools teamens-resolver-keys.json
code review is required from DevTools team