Merge pull request #39 from allo-protocol/fix/wrong-comments

fix: wrong comments
allo-protocol · Oct 21, 2024 · c583e0e · c583e0e
2 parents d39e157 + 9f24fe6
commit c583e0e
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/contracts/core/Allo.sol b/contracts/core/Allo.sol
@@ -72,6 +72,7 @@ contract Allo is IAllo, Initializable, Ownable, AccessControlUpgradeable, Reentr
 
     /// @notice Maps the pool ID to the pool details
     /// @dev 'Pool.id' -> 'Pool'
+    /// @dev pool ID is incremental and starts from 1, there is no _pools[0]
     mapping(uint256 => Pool) internal _pools;
 
     /// @custom:oz-upgrades-renamed-from cloneableStrategies

diff --git a/contracts/core/Anchor.sol b/contracts/core/Anchor.sol
@@ -53,7 +53,7 @@ contract Anchor is ERC721Holder, ERC1155Holder {
     /// ==========================
 
     /// @notice Constructor
-    /// @dev We create an instance of the 'Registry' contract using the 'msg.sender' and set the profileId.
+    /// @dev We create an instance of the 'Registry' contract using the _registry and _profileId.
     /// @param _profileId The ID of the allowed profile to execute calls
     constructor(bytes32 _profileId, address _registry) {
         registry = Registry(_registry);

diff --git a/contracts/core/Registry.sol b/contracts/core/Registry.sol
@@ -300,7 +300,7 @@ contract Registry is IRegistry, Initializable, AccessControlUpgradeable, Errors
     }
 
     /// @notice Removes members from the profile
-    /// @dev 'msg.sender' must be the pending owner of the profile.
+    /// @dev 'msg.sender' must be the owner of the profile.
     /// @param _profileId The ID of the profile
     /// @param _members The members to remove
     function removeMembers(bytes32 _profileId, address[] memory _members) external onlyProfileOwner(_profileId) {
@@ -377,7 +377,7 @@ contract Registry is IRegistry, Initializable, AccessControlUpgradeable, Errors
         return hasRole(_profileId, _member);
     }
 
-    /// @notice Transfers any fund balance in Allo to the recipient
+    /// @notice Transfers any fund balance in Registry to the recipient
     /// @dev 'msg.sender' must be the Allo owner
     /// @param _token The address of the token to transfer
     /// @param _recipient The address of the recipient