search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

setappointment success message clarity #1170

Open nus-se-script opened 1 week ago

nus-se-script commented 1 week 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"

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

SuhailLoya commented 1 week ago

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

Duplicate status (if any):

--