nus-cs2113-AY2425S1 / pe-dev-response

0 stars 0 forks source link

delete-water uses (water ml) for input instead of index #113

Open nus-pe-bot opened 2 days ago

nus-pe-bot commented 2 days ago

image.png


[original: nus-cs2113-AY2425S1/pe-interim#215] [original labels: severity.Medium type.DocumentationBug]

ayushi0803 commented 1 day ago

Team's Response

Thank you for reporting this issue. We acknowledge that the delete-water command currently uses the water amount (ml) as input instead of an index. This is a documentation discrepancy, as the command is meant to delete a specific water entry by index, not by its amount.

We accept this feedback and plan to update the documentation to reflect the correct usage of the delete-water command. However, we would like to clarify that this issue is related to documentation, and the functionality itself is not affected. As such, we have lowered the severity to Low.

Thank you for your understanding, and we appreciate your feedback!

Duplicate status (if any):

--