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
The check Ownable.assert_only_owner(); shouldn't be in the library because it belongs to a contract logic (who is the caller) and not a library logic (what is the function actually doing).
This issue is consequently about moving all these checks (not only only_owner but similar semantic) at the contract level
The text was updated successfully, but these errors were encountered:
In a lot of places, we currently do in the contract
and in the associated library
The check
Ownable.assert_only_owner();
shouldn't be in the library because it belongs to a contract logic (who is the caller) and not a library logic (what is the function actually doing).This issue is consequently about moving all these checks (not only only_owner but similar semantic) at the contract level
The text was updated successfully, but these errors were encountered: