ty4g1 / pe

0 stars 0 forks source link

Clear command does not impose format #12

Open ty4g1 opened 1 week ago

ty4g1 commented 1 week 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

nus-pe-script commented 4 days 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

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]


## :question: Issue type Team chose [`type.DocumentationBug`] Originally [`type.FeatureFlaw`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]