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.

Sign up for GitHub

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

Jump to bottom

[CS2113-F14-2] DinEZ #82

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

[CS2113-F14-2] DinEZ #82

wants to merge 432 commits into from

Conversation

Zhengwinter
Copy link

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
@samuelory
Merge pull request nus-cs2113-AY2324S2#82 from JeffinsonDarmawan/Jeff…
f8dca7e 
…inson-UML

DG2
adamzzq and others added 17 commits March 31, 2024 23:21
@adamzzq
Merge pull request #82 from adamzzq/branch-print-receipt
75230bd 
Implement getReceipt method in Order.java
@adamzzq
Implement Order.getReceipt method with discount
6ad59e5
@adamzzq
Merge pull request #84 from adamzzq/branch-print-receipt
b4e9002 
Implement Order.getReceipt method with discount
@Zhengwinter
Implement Menu commands
6f5ba80
@Zhengwinter
Merge branch 'master' into master
35938ea
@Zhengwinter
Restructure the complete command to be used by both order and menu logic
d7e1a94
@Zhengwinter
Resolve merge conflicts
2062fec
@Zhengwinter
Reformat code to comply with char limit
e180aee
@Zhengwinter
Merge pull request #85 from Zhengwinter/master
d2efbed 
Implement Menu commands (MenuAdd, MenuDelete, MenuComplete)
@Zhengwinter
Refactor code and abstract out classes, format in CamelCase
7927ae7
@Zhengwinter
Merge pull request #86 from Zhengwinter/master
afbbea0 
Refactor code and abstract out classes, format in CamelCase
@Xb990219
Update V2.0 working draft
a61ab35
@Xb990219
Merge branch 'Xb990219-mainLogic'
f8fc688
@Xb990219
resolve merge conflict
578139d
@Xb990219
Fix minor bugs
cc61ff8
@Xb990219
Fix checkstyle
8d19008
@Xb990219
Merge pull request #88 from Xb990219/master
8fccca2 
Update V2.0
ThawTunZan
ThawTunZan reviewed Apr 3, 2024
View reviewed changes
Copy link

@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

ThawTunZan
ThawTunZan reviewed Apr 3, 2024
View reviewed changes
Copy link

@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

ThawTunZan
ThawTunZan reviewed Apr 3, 2024
View reviewed changes
Copy link

@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

ThawTunZan
ThawTunZan reviewed Apr 3, 2024
View reviewed changes
Copy link

@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
@webtjs
Change image source file in DG
e517e0b
@webtjs
Merge pull request #194 from webtjs/master
75eace8 
Update DG
@webtjs
Fix formatting issue in DG
526f4e5
@webtjs
Merge pull request #195 from webtjs/master
775c6d7 
Fix formatting issue in DG
@webtjs
Merge branch 'master' into branch-webtjs-PPP
f239d0c
@adamzzq
Update Zheqi's PPP
6a150cb
@adamzzq
Merge pull request #196 from adamzzq/branch-ppp
38c7123 
Update Zheqi's PPP
@adamzzq
Add new User Stories in DG, and correct some grammatical errors
88e1410
@adamzzq
Merge branch 'master' of https://github.com/AY2324S2-CS2113-F14-2/tp …
85196f7 
…into branch-more-user-stories
@adamzzq
Merge pull request #197 from adamzzq/branch-more-user-stories
a3de7ed 
Branch more user stories
@adamzzq
Update Zheqi's PPP
2aa9de7
@adamzzq
Update Zheqi's PPP
56fcc62
@adamzzq
Merge pull request #198 from adamzzq/branch-ppp
4191744 
Update Zheqi's PPP
@webtjs
Update webtjs PPP
e4e69e3
@webtjs
Merge branch 'master' into branch-webtjs-PPP
943aa10
@webtjs
Update LoC in webtjs PPP
e901e41
@Zhengwinter
Edit UG and fix grammatical errors and formatting issues
047c643
@Zhengwinter
Merge pull request #200 from Zhengwinter/master
d359765 
Edit UG and fix grammatical errors and formatting issues
@webtjs
Merge pull request #199 from webtjs/branch-webtjs-PPP
8ffa4aa 
Update webtjs PPP
@Zhengwinter
Comment out loggers and update DG
661bd43
@webtjs
Update link in webtjs PPP
930bfb2
@Zhengwinter
Remove unused imports
573a685
@webtjs
Merge pull request #202 from webtjs/branch-webtjs-PPP
b7ee53d 
Update link in webtjs PPP
@Zhengwinter
Merge pull request #201 from Zhengwinter/master
ba1b82a 
Comment out loggers and update DG
@Zhengwinter
Update Wentao and Xiao Bo PPP
1e5d4d2
@Zhengwinter
Merge pull request #203 from Zhengwinter/master
9fe969c 
Update Wentao and Xiao Bo PPP
@adamzzq
Update Zheqi's PPP
74d5a8a
@adamzzq
Merge pull request #204 from adamzzq/branch-ppp
a1f2812 
Update Zheqi's PPP
@Zhengwinter
Minor update to PPP
febade4
@Zhengwinter
Merge pull request #205 from Zhengwinter/master
9b549e9 
Minor update to PPP
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@royaliceblue royaliceblue royaliceblue left review comments

@ThawTunZan ThawTunZan ThawTunZan left review comments

@NGXZS NGXZS NGXZS left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@Zhengwinter @royaliceblue @ThawTunZan @NGXZS @adamzzq @Xb990219 @webtjs