L5-Z / pe

0 stars 0 forks source link

Error message unclear (`smart`) #1

Open L5-Z opened 7 months ago

L5-Z commented 7 months ago
smart
Here is a smart itinerary for your date (the afternoon is planned around your base preferences!):
We begin with lunch at Westside Wraps, followed by some fun at Jurong Hill Lookout Tower.
We proceed to have dinner at Spicy Dragon, and finish the night at Siloso Beach Bars Evening Out.

Are you satisfied with the date idea? [Yes/No]
N
Sorry, I didn't quite understand that :(
Please enter either yes or no or cancel the generation process
____________________________________________________________

Instructions on either yes, no or cancel are unclear. Error message structure does not properly convey that cancel will cancel the generation process. Users may end up typing cancel the generation process though as ridiculous as it may sound.

It was done correctly in other instances such as:

____________________________________________________________
list
Looking for ideas to spice up your date night?
Choose from the following options:
1. List out delicious dining options (type 'food')
2. Discover exciting activities to do together (type 'activities')
3. Unwrap joy with our curated list of gifts that'll make hearts flutter! (type 'gifts')
4. Changed your mind? Feel free to cancel this command! (type 'cancel')
What's your pleasure?
____________________________________________________________
2
Invalid option! Please choose 'food', 'activities', 'gifts' or 'cancel'.
nus-pe-bot commented 7 months ago

Team's Response

We thought that it's okay because it was clear that only a "Yes" or "No" was expected due to the square brackets right next to the question, but agree that the error message could be made clearer! Screenshot 2024-04-20 at 2.31.53 PM.png

Items for the Tester to Verify

:question: Issue type

Team chose [type.FunctionalityBug] Originally [type.DocumentationBug]

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [x] I disagree **Reason for disagreement:** While it may be categorised as a typo, I feel that in the instance of the user wanting to cancel, they may be confused with the instructions in the error message. So it ends up being not so much a cosmetic defect but may actually give rise to misinterpretation and faulty usage attempts of the app, thus making it a Low severity.