Skip to content

[CS2113-F14-2] DinEZ#82

Open
Zhengwinter wants to merge 432 commits intonus-cs2113-AY2324S2:masterfrom
AY2324S2-CS2113-F14-2:master
Open

[CS2113-F14-2] DinEZ#82
Zhengwinter wants to merge 432 commits intonus-cs2113-AY2324S2:masterfrom
AY2324S2-CS2113-F14-2:master

Conversation

@Zhengwinter
Copy link
Copy Markdown

The application is an integrated system solution which aims to help operations manager address all aspects of restaurant management that could be encountered during the daily running of a restaurant in a digitalized way, including order taking, delivery tracking, inventory monitoring and customer feedback etc.

JeffinsonDarmawan pushed a commit to JeffinsonDarmawan/tp that referenced this pull request Mar 29, 2024
Copy link
Copy Markdown

@ThawTunZan ThawTunZan left a comment

Choose a reason for hiding this comment

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

Add a example output for each commands

Copy link
Copy Markdown

@ThawTunZan ThawTunZan left a comment

Choose a reason for hiding this comment

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

There is "OrderLogic", but may want to include the Logic for other actions like remove or view

Copy link
Copy Markdown

@ThawTunZan ThawTunZan left a comment

Choose a reason for hiding this comment

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

May want an explanation for some of the important methods in each class

Copy link
Copy Markdown

@ThawTunZan ThawTunZan left a comment

Choose a reason for hiding this comment

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

May want to make it neater with the headers to make it easier to read

webtjs and others added 30 commits April 15, 2024 20:25
Fix formatting issue in DG
Edit UG and fix grammatical errors and formatting issues
Comment out loggers and update DG
Update Wentao and Xiao Bo PPP
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants