Skip to content
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 author tags, removed unused codes and updated PPP and UG and DG #214

Merged
merged 11 commits into from Nov 12, 2018

Conversation

SalsabilTasnia
Copy link
Collaborator

No description provided.

@coveralls
Copy link

coveralls commented Nov 12, 2018

Coverage Status

Coverage increased (+0.2%) to 80.807% when pulling a9abdf6 on SalsabilTasnia:version3 into 30777a0 on CS2113-AY1819S1-F09-2:master.

@SalsabilTasnia SalsabilTasnia added the status.review-pending The pull request is ready to be reviewed. label Nov 12, 2018
@px1099 px1099 added type.task Something that needs to be done, but not a story, bug, or an epic. priority.high Must do labels Nov 12, 2018

==== Delete Menu

Delete an item on the menu +
Format: `deletemenu INDEX`

* Deletes the menu item in that particular index
* Note that the user must perform 'List Menu' , 'List Menu According to Category' or 'Find Menu' first before performing 'Delete Menu'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can put this under pre-conditions instead

@kianhong95 kianhong95 merged commit 4c93d5d into CS2113-AY1819S1-F09-2:master Nov 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority.high Must do status.review-pending The pull request is ready to be reviewed. type.task Something that needs to be done, but not a story, bug, or an epic.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants