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

[F09-2] Restaurant Management System #162

Open
wants to merge 870 commits into
base: master
Choose a base branch
from

Conversation

kangmingtay
Copy link

No description provided.

@stephlewyh
Copy link

stephlewyh commented Oct 19, 2018

Dear team F09-2,

Your team has shown good demo of your product with code updates.
However you have not updated your developer guide and user guide accordingly. As mentioned on the project website, you are recommended to do this incrementally each week.

Some housekeeping points you should note moving ahead:

Developer Guide:
Each team member should describe at least 1 feature with a UML diagram eg. Activity, Sequence, Component diagram etc. As for UML diagrams, they should not include too low-level details or be too big, as that would go against the top-down approach for documentation. You may refer to comments I have made for the other teams regarding UML diagrams in the developer guide.

Project Management:
To view your own progress, you could also go to your team repo page and click 'project insights' tab for team statistics, aside from using the issue tracker and milestones to help pace your progress.

kangmingtay and others added 30 commits November 12, 2018 22:13
EmployeeEmail now extends Email and EmployeePhone regex has been changed
update menu name limitations and UG
made some changes to member section
Update README and ContactUs
fix inconsistencies in PR
Updated constraints for Add Employee Command
Update ui.png and version number
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants