search

AaronCQL / pe

0 stars 0 forks source link

Confusing error message for reminder command #8

Open AaronCQL opened 4 years ago

AaronCQL commented 4 years ago

With a wrong usage of parameters like reminder 1 days/23 hours/1:

Screenshot_20200417_145329.png

The use of "reminder date should be within one week from now" does not really explain how the days/ parameter should be used.

nus-pe-bot commented 4 years ago

Team's Response

The UG has stated that days how to use the reminder Command. Read the UG or type help, thank you.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I fully agree that the UG does indeed explain how to use the reminder command. However, should a user accidentally enter a wrong number for /days (which can arguably happen sometimes), the error message shown is not descriptive enough for the user to understand where exactly he went wrong. For instance, "date should be within one week from now" seems to hint to the user that the app expects the actual date (like 20 Feb 2020) within the week. Furthermore, the inclusion of the "integer" also confuses the user (eg. how can a date also be an integer?).

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: The team has not justified why they reduced the severity. Moreover, this bug falls in line with the severity.Low description that it "only appears in very rare situations and causes a minor inconvenience only".