Where ? : in contracts
What happened ?: The contract mixes string requires ("Recipient address is zero") with custom errors.
What should be?: Replace all require strings with custom errors (like, error InvalidRecipient(), error PaymentFailed())
Why?: Easy deployment and efficient contract because Strings cost more gas to deploy and execute.
@kumawatkaran523 I have Proposed a fix Please review, #102
Where ? :
in contractsWhat happened ?: The contract mixes string requires ("Recipient address is zero") with custom errors.
What should be?: Replace all require strings with custom errors (like, error
InvalidRecipient(), errorPaymentFailed())Why?: Easy deployment and efficient contract because Strings cost more gas to deploy and execute.
@kumawatkaran523 I have Proposed a fix Please review, #102