muller317 / pe

0 stars 0 forks source link

No warnings for clear command #7

Open muller317 opened 1 week ago

muller317 commented 1 week ago

There is no warnings for entering the command clear, this may be unlikely to happen but the outcome may be disastrous if such input was to be made accidentally.

nus-pe-script commented 1 week ago

Team's Response

In UG: Screenshot 2024-11-17 at 11.46.32 PM.png Screenshot 2024-11-17 at 11.46.41 PM.png Screenshot 2024-11-17 at 11.46.48 PM.png

In Product Help Window: Screenshot 2024-11-17 at 11.47.11 PM.png

Thank you for your feedback! We understand that removing all entries could be disastrous for the user if the action was unintended. Hence as provided above, we did notify the user to note that clear action is irreversible under the quick start section and under the description for the clear command feature too. To inform and ensure the user knows that this action is irreversible.

Under Data Management section we also encourage user to make back up just in case such disastrous event occur.

Moreover, we also note the user in the in app help message that student records that cleared by cleared command is not recoverable, and the help message will pop up in the first use of the application too.

As mentioned that accidentally using command clear is unlikely to happen, we believe this could be a severity.Low.

Having a warning or confirmation check before command that remove student detail such as clear or delete is a good suggestion. We would definitely consider implement it in the coming future, hence our team would consider this as a NotInScope bug.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.High`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]