nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Error message unclear (`smart`) #2528

Open nus-se-script opened 6 months ago

nus-se-script commented 6 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'.

[original: nus-cs2113-AY2324S2/pe-interim#2500] [original labels: severity.Low type.DocumentationBug]

soongensayo commented 6 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

Duplicate status (if any):

--