SIP-7 Specify error type for substandard validation failures

[frankisawesome]

I think it makes sense for the SIP to specify the error type returned when the zone fails to validate a particular substandard check.

A good one could be SubstandardViolation(uint8 substandardId, string reason, bytes32 orderHash

[ryanio]

We already have error InvalidSubstandard(uint8 substandard) in SIP-10. I like your idea but harcoding reasons might be too verbose and cost too much in terms of contract size (similar to old school string revert reasons compared to custom errors). So better custom errors might be a preferred approach. Adding orderHash seems smart too :)

[frankisawesome]

Yep fair enough - I will need to read up on SIP-10 and will close this now :)