Functions mutating storage should emit an Event for easy off-chain monitoring.
🤦 Bad:
function setFee(uint256 _fee) external onlyOwner {
fee = _fee;
}
🚀 Good:
function setFee(uint256 _fee) external onlyOwner {
emit ChangedFee(fee, _fee);
fee = _fee;
}