comments cleanup on ContractMetadataregistry

Author: Krishang Nadgauda

contracts/ContractMetadataRegistry.sol

@@ -10,32 +10,37 @@ import "@openzeppelin/contracts/metatx/ERC2771Context.sol";
 import { IContractMetadataRegistry } from "./interfaces/IContractMetadataRegistry.sol";
 
 contract ContractMetadataRegistry is IContractMetadataRegistry, ERC2771Context, Multicall, AccessControlEnumerable {
+    
+    /// @dev Only accounts with OPERATOR_ROLE can register metadata for contracts.
     bytes32 public constant OPERATOR_ROLE = keccak256("OPERATOR_ROLE");
 
     /*///////////////////////////////////////////////////////////////
-                            State variables
+                            Mappings
     //////////////////////////////////////////////////////////////*/
 
     /// @dev contract address deployed => metadata uri
     mapping(address => string) public getMetadataUri;
 
     /*///////////////////////////////////////////////////////////////
-                    Constructor + modifiers
+                            Constructor
     //////////////////////////////////////////////////////////////*/
 
     constructor(address _trustedForwarder) ERC2771Context(_trustedForwarder) {
         _setupRole(DEFAULT_ADMIN_ROLE, _msgSender());
     }
 
     /*///////////////////////////////////////////////////////////////
-                            External methods
+                        External functions
     //////////////////////////////////////////////////////////////*/
 
+    /// @dev Records `metadataUri` as metadata for the contract at `contractAddress`.
     function registerMetadata(address contractAddress, string memory metadataUri) external {
-        require(hasRole(OPERATOR_ROLE, _msgSender()), "not operator.");
+        require(hasRole(OPERATOR_ROLE, _msgSender()), "Not operator.");
         require(bytes(metadataUri).length > 0, "No metadata");
         require(bytes(getMetadataUri[contractAddress]).length == 0, "Metadata already registered");
+        
         getMetadataUri[contractAddress] = metadataUri;
+        
         emit MetadataRegistered(contractAddress, metadataUri);
     }
 

contracts/interfaces/IContractMetadataRegistry.sol

@@ -5,5 +5,6 @@ interface IContractMetadataRegistry {
     /// @dev Emitted when a contract metadata is registered
     event MetadataRegistered(address indexed contractAddress, string metadataUri);
 
+    /// @dev Records `metadataUri` as metadata for the contract at `contractAddress`.
     function registerMetadata(address contractAddress, string memory metadataUri) external;
 }

docs/ContractMetadataRegistry.md

@@ -35,7 +35,7 @@ function OPERATOR_ROLE() external view returns (bytes32)
 
 
 
-
+*Only accounts with OPERATOR_ROLE can register metadata for contracts.*
 
 
 #### Returns
@@ -225,7 +225,7 @@ function registerMetadata(address contractAddress, string metadataUri) external
 
 
 
-
+*Records `metadataUri` as metadata for the contract at `contractAddress`.*
 
 #### Parameters
 

docs/IContractMetadataRegistry.md

@@ -18,7 +18,7 @@ function registerMetadata(address contractAddress, string metadataUri) external
 
 
 
-
+*Records `metadataUri` as metadata for the contract at `contractAddress`.*
 
 #### Parameters