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
added delete reminder command and test cases #113
added delete reminder command and test cases #113
Conversation
|
||
public static final String COMMAND_WORD = "reminder"; | ||
|
||
public static final String MESSAGE_USAGE = COMMAND_WORD + ": Adds a Reminder to MoneyGoWhere. " | ||
public static final String MESSAGE_USAGE = COMMAND_WORD + " " + AddReminderCommand.COMMAND_WORD |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can probably reference AddReminderCommand's usage here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for pointing out
} | ||
+ PREFIX_MESSAGE + "MESSAGE " | ||
+ "\n" + " or " + "\n" | ||
+ COMMAND_WORD + " delete" + ": Remove a Reminder from MoneyGoWhere. " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same for delete
|
||
public static final String MESSAGE_SUCCESS = "New Reminder added: %1$s"; | ||
|
||
private final Reminder toAdd; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Better to keep it as reminder
or value
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i will change the variable name to reminderToAdd.
Fixes #127 |
Aded delete reminder command and test cases