Skip to content

Commit

Permalink
Fix missing ERC-7201 natspec tag (L-05) (#82)
Browse files Browse the repository at this point in the history
* Add in missing ERC7201 natspec tag

* Addressed review comment

* Corrected hash of GovernorSequentialProposalIdStorageLocation
  • Loading branch information
jferas authored Dec 16, 2024
1 parent 3dfdd7e commit 20f51ed
Showing 1 changed file with 3 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ abstract contract GovernorSequentialProposalIdUpgradeable is GovernorUpgradeable
bytes32 descriptionHash;
}

/// @custom:storage-location erc7201:storage.GovernorSequentialProposalIdStorage
struct GovernorSequentialProposalIdStorage {
/// @notice The next proposal ID to assign to a proposal.
uint256 _nextProposalId;
Expand All @@ -30,10 +31,10 @@ abstract contract GovernorSequentialProposalIdUpgradeable is GovernorUpgradeable
mapping(uint256 proposalId => ProposalDetails) _proposalDetails;
}

// keccak256(abi.encode(uint256(keccak256("openzeppelin.storage.GovernorSequentialProposalIdStorage")) - 1)) &
// keccak256(abi.encode(uint256(keccak256("storage.GovernorSequentialProposalIdStorage")) - 1)) &
// ~bytes32(uint256(0xff))
bytes32 private constant GovernorSequentialProposalIdStorageLocation =
0xa6952339bc887ea688c6b8e8399bb953a5002ee79177d6322fca98dc89ae0b00;
0x357e1d0c89980520b3654c57f444238d75a15e5f41d389a090caabe54617d800;

function _getGovernorSequentialProposalIdStorage()
private
Expand Down

0 comments on commit 20f51ed

Please sign in to comment.