b1llp4rk / pe Goto Github PK

suggestion: The index must be a positive integer from 1 to 2,147,483,647, inclusive.

This is shorter and more concise

The error message should guide or point out to the user which part of the command was wrong. The current error message does give some information on reminders command, but nowhere in the UG nor the error message state that past dates are invalid.

this is a dummy bug

edited a dummy bug

readAddressBook() is not a static method, so the method call should be to a JsonAddressBookStorage object.
This means that there should be a colon : before JsonAddressBookStorage.

The UG states

The following is an extract from quick start

help and "help" in 6 and 7 should have been help (bolded).

The error message given to the below command
remind rd/2022-04-16 1 r/have dinner
is

A wrong command format input should result in AIA to give a message about how to use the command correctly.

However the example given is not accurate.

When i use the given example command remind 2 r/meeting rd/01 Jan 2022, the response of AIA is

description should be DESCRIPTION, to adhere to the rules stated earlier, which is

When all people are listed using list, reminders rd/ command finds all the reminders for today.

However when I use find and only some people are listed, the reminders rd/ command only find reminders for the peeple who are displayed

before execution

after execution

The expected behaviour is to show all reminders for all person regardless of who is currently displayed in the list.