Weather Witness

First Flight #40
Beginner FriendlyFoundrySolidityNFT
100 EXP
View results
Submission Details
Impact: medium
Likelihood: low
Invalid

Lack of setter functions for mint price parameters `s_currentMintPrice` and `s_stepIncreasePerMint`

Root + Impact

Description

  • The contract does not expose setter functions for either s_currentMintPrice or s_stepIncreasePerMint to control mint pricing and price increments per mint. These parameters are crucial for maintaining fair and adaptable pricing over time.

  • This limits the owner’s ability to adjust pricing if it's initially set too high or too low, potentially impacting user participation and fairness.


Likelihood:

  • Changing market conditions could require price adjustments.

Impact:

  • Unadjustable prices may reduce engagement and fairness.

Recommended Mitigation

+ function updateCurrentMintPrice(uint256 currentMintPrice) external onlyOwner {
+ s_currentMintPrice = currentMintPrice;
+ }
+ function updatesStepIncreasePerMint(uint256 stepIncreasePerMint) external onlyOwner {
+ s_stepIncreasePerMint = stepIncreasePerMint;
+ }
Updates

Appeal created

bube Lead Judge 5 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.