AdiMangalam / pe

0 stars 0 forks source link

Integrate a success message for Delete command #6

Open AdiMangalam opened 1 week ago

AdiMangalam commented 1 week ago

image.png

Delete -r - doesn't have a successMessage, but rest of the 'Delete' commands have one. It is suggest to have a uniform code and add a success message to Delete -r as well.

nus-se-script commented 6 days ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Deleting a row of entries shows no success message.

image.png

When "delete -r - is entered, though the command works well, the success message is missing.


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

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

We are aware and we accept this as a bug. However we feel that the missing success message for the "delete -r" command is a low severity bug because it does not affect the functionality of the command, which works as intended. With the current design of Inventra delete function even if success message is printed, users will still need to verify records deleted using the view -a command, making this a minor issue with no impact on usability.

We deem this a duplicate with #167 as it is the same issue of no success message for delete -r.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


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