nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Missing Detail in Error Message for `addInsurance` #3828

Open nus-se-script opened 1 week ago

nus-se-script commented 1 week ago

The error message displayed when the user enters an invalid insurance id for the addInsurance command is:

This Insurance Plan ID is invalid!

This message does not provide the user with knowledge about "what is a valid Insurance Plan ID?" It is recommended to include the valid Insurance Plan IDs within the error message, so users do not have to look up the user guide every time.

E.g. Please enter a valid Insurance Plan ID! (0 - Basic Insurance Plan, 1 - Travel Insurance Plan)

image.png


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

RezwanAhmed123 commented 1 week ago

Team's Response

Thanks for raising this concern. However, we feel the message is meant to be general and concise so that in future when more plans are added the message does not look congested. Frequent users of the app should know what the insurance plans they use often are and the user guide is meant to guide properly in that respect anyways.

Duplicate status (if any):

--