nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Clear command does not impose format #695

Open nus-pe-bot opened 2 weeks ago

nus-pe-bot commented 2 weeks ago

The clear command allows users to enter words after clear, and executes the command successfully.

Would be better to impose a format check, in case a new user runs clear owners or clear pets and only expects that section to be cleared.

image.png


[original: nus-cs2103-AY2425S1/pe-interim#981] [original labels: type.FeatureFlaw severity.Low]

Runus01 commented 1 week ago

Team's Response

Thank you for the feedback. Actually, this is expected behaviour according to our UG which states that 'clear' command will clear all pets and owners from the database. Typing anything after a command will still have the same result as the command itself. There is a warning also that states that the entire data file will be lost and a copy will be needed. We will improve the intuitiveness of the 'clear' command in a future iteration as mentioned in the planned enhancements and will also update the UG to make it clear to new users that "clear owners" or "clear pets" has the same effect as "clear".

photo_2024-11-17_23-26-24.jpg

image.png

Duplicate status (if any):

--