search

nus-cs2103-AY1920S1 / pe-dev-response

0 stars 0 forks source link

Set reminder shows error #162

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

Expected Behavior

Reminder should be set.

Current Behavior

Error is shown, and to be honest, user has no idea what to do after that (in my case, I did not get an idea for what setReminder actually does`). More details needed in documentation.

Steps to Reproduce

  1. addReminder n/Don’t be broke. l.b/10 u.b/20 start/2019/01/01 end/2019/12/30
  2. listReminders
  3. setReminder 1 typ/expense n/Mala p/1d f/daily

Screenshot 2019-11-15 at 12.57.58 PM.png

[original: nus-cs2103-AY1920S1/pe-interim#162]

yuhongtay commented 4 years ago

Team's Response

SetReminder allows the user to set a reminder that sends notification when the dateline of the reminder draws near. Similar to google calendar reminders. For instance, user creates an expense at 20/11/2019,

addExpense cat/Food n/Mala amt/5.50 d/2019 11 20 tg/food tg/indulgence

and sets a reminder setReminder 1 typ/expense n/Mala p/2d f/daily

Where '1' is the index of said expense in the expense pannel. The user will receive a notification about said event on 2019 11 18, (as specified by p/2d which means 2 days in advance), daily (as specified by f/daily).

Reminders cannot be set for events that already happened or event on the current day itself (that is, 15 Nov during the PE).

The local date stored in the program will be the current date from the system clock in the default time-zone. However, for testing purposes, you can use the command "setDate YYYY MM DD" to temporarily set the date in the program.

e.g. type: setDate 2019 11 18 sets the date to 2019 11 18. setDate sets the date back to the current date.

Duplicate status (if any):

--