QA Report #94
Labels
bug
Something isn't working
duplicate
This issue or pull request already exists
QA (Quality Assurance)
Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax
wontfix
out of scope, a non-issue, or something already addressed
[L01] Add 2 step Changes for Critical Changes
Critical changes such as ownership updates should be a 2 step process to protect against human error. While the errors are unlikely important parts of the contract would become unusable if they occured.
Consider changing the following functions to 2 step procedures.
Swivel.sol#L428
MarketPlace.sol#L53
Creator.sol#L47
[L02] Unlocked Pragma
Recommend not using a floating pragma and changing to 0.8.13 to be consistent with other contracts.
ZcToken.sol#L2
[N01] Open Todos
There are 17 open todos throughout swivel.sol, recommend resolving and removing before deployment.
The text was updated successfully, but these errors were encountered: