QA Report

[code423n4]

UNUSED STATE VARIABLE SHOULD BE REMOVED

Impact - NON CRITICAL

File: contracts/Project.sol

line 60 : uint256 public constant override VERSION = 25000;

CONSTANTS SHOULD BE DEFINED RATHER THAN USING MAGIC NUMBERS

Impact - NON CRITICAL

File: contracts/libraries/SignatureDecoder.sol

line 25 : if (messageSignatures.length % 65 != 0)
line 35 : if (v != 27 && v != 28)
line 82 : if (v < 27)

PUBLIC FUNCTIONS NOT CALLED BY THE CONTRACT SHOULD BE DECLARED EXTERNAL INSTEAD

Impact - NON CRITICAL

File: contracts/Community.sol

line 700 : function isTrustedForwarder(address _forwarder) public view override(ERC2771ContextUpgradeable, ICommunity) returns (bool)

File: contracts/DebtToken.sol

line 82 : function decimals() public view virtual override returns (uint8)

File: contracts/Disputes.sol

line 187 : function isTrustedForwarder(address _forwarder) public view override(ERC2771ContextUpgradeable, ICommunity) returns (bool)

File: contracts/HomeFi.sol

line 264 : function isTrustedForwarder(address _forwarder) public view override(ERC2771ContextUpgradeable, ICommunity) returns (bool)

File: contracts/Project.sol

line 727 : function isTrustedForwarder(address _forwarder) public view override(ERC2771ContextUpgradeable, ICommunity) returns (bool)

File: contracts/ProjectFactory.sol

line 98 : function isTrustedForwarder(address _forwarder) public view override(ERC2771ContextUpgradeable, ICommunity) returns (bool)