-
Notifications
You must be signed in to change notification settings - Fork 4
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
Fix bugs raised in PE-D #134
Fix bugs raised in PE-D #134
Conversation
Sorry for laziness earlier and not specifying the issues I targeted, but it's all the issues assigned to me that have been resolved. You can filter the issue list and get a comprehensive view |
@@ -2,22 +2,22 @@ | |||
|
|||
import static seedu.address.logic.parser.ParserUtil.parseNoArgs; | |||
|
|||
import seedu.address.logic.commands.ShowFavouriteCommand; | |||
import seedu.address.logic.commands.ListFavouriteCommand; | |||
import seedu.address.logic.parser.exceptions.ParseException; | |||
|
|||
/** | |||
* Parses input arguments and creates a new ShowFavouriteCommand object |
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.
Documentation not changed to new commands yet
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.
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.
LGTM on the documentation changes!
Fixed the bugs explicitly assigned to me so far.
Might add more commits for further bugs.