You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
Some of our contracts still use require, instead of custom error messages. As pointed out by @DevABDee in #7, this can lead to higher gas fees than necessary. More info can be found here.
Describe the solution you'd like
Clean our contracts by introducing custom error messages wherever it is beneficial. Please provide relevant screenshots on performed tests and compare before <--> after gas values.
Additional context
For questions, please ask here or join the discussion on Discord.
The text was updated successfully, but these errors were encountered:
Is your feature request related to a problem? Please describe.
Some of our contracts still use
require
, instead of custom error messages. As pointed out by @DevABDee in #7, this can lead to higher gas fees than necessary. More info can be found here.Describe the solution you'd like
Clean our contracts by introducing custom error messages wherever it is beneficial. Please provide relevant screenshots on performed tests and compare before <--> after gas values.
Additional context
For questions, please ask here or join the discussion on Discord.
The text was updated successfully, but these errors were encountered: