search

shuckycheese / pe

0 stars 0 forks source link

setappointment success message clarity #10

Open shuckycheese opened 4 days ago

shuckycheese commented 4 days ago

image.png

When deleting an appointment from a person using the command setappointment 1 d/ the success message says the above. The success message is not very clear

Maybe you could display another success message for when the value to the prefix d/ is empty saying something like "appointment successfully removed"

soc-se-bot commented 11 hours ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

Valid point, we agree. But the current message still makes sense, that is the date was removed.

Our group didn't feel this was important enough to be fixed by v1.6 since there were more pressing issues to handle and user could still use the product meaningfully with the current error message. Normal operations are not affected and there is no inconvenience caused to the user.

Classified as not in scope

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your reason]