Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PE-D] Error messages are slightly too general #182

Closed
nus-pe-bot opened this issue Oct 30, 2020 · 1 comment
Closed

[PE-D] Error messages are slightly too general #182

nus-pe-bot opened this issue Oct 30, 2020 · 1 comment
Labels
wontfix This will not be worked on

Comments

@nus-pe-bot
Copy link

Error messages are currently shown for invalid commands, displaying the proper syntax as well as examples which is very helpful. It would really boost user experience if the specific error was mentioned.
e.g. The following command was entered: add n/ben chan p/1234 s/good school y/p1 v/home t/8 1000-1200 f/10 d/24/10/10
Result:
image.png

As seen in the above picture, it would be slightly better if the error message was that the day_of_week field was out of bounds.


Labels: severity.Low type.FeatureFlaw
original: EkChinHui/ped#7

@ong-yinggao98 ong-yinggao98 added the wontfix This will not be worked on label Nov 6, 2020
@ong-yinggao98
Copy link

day is handled with time so the error message should be the same

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants