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

[CS2103T-T12-1] OneShelf #19

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

Conversation

Wincenttjoi
Copy link

OneShelf is an app designed to help restaurant businesses manage their inventory stocks which translates to higher efficiency and reduces the chances of over-stocking. OneShelf is most compatible with users who are fast-typist.

@codecov-commenter
Copy link

Codecov Report

Merging #19 into master will decrease coverage by 4.87%.
The diff coverage is 68.30%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #19      +/-   ##
============================================
- Coverage     72.31%   67.43%   -4.88%     
+ Complexity      400      366      -34     
============================================
  Files            70       69       -1     
  Lines          1228     1216      -12     
  Branches        124      119       -5     
============================================
- Hits            888      820      -68     
- Misses          308      349      +41     
- Partials         32       47      +15     
Impacted Files Coverage Δ Complexity Δ
...n/java/seedu/address/commons/core/GuiSettings.java 69.23% <ø> (ø) 5.00 <0.00> (ø)
...main/java/seedu/address/commons/core/Messages.java 0.00% <ø> (ø) 0.00 <0.00> (ø)
...ain/java/seedu/address/logic/commands/Command.java 100.00% <ø> (ø) 1.00 <0.00> (ø)
.../seedu/address/logic/parser/FindCommandParser.java 100.00% <ø> (ø) 3.00 <0.00> (ø)
.../model/item/exceptions/DuplicateItemException.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
...s/model/item/exceptions/ItemNotFoundException.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
.../java/seedu/address/model/util/SampleDataUtil.java 27.27% <0.00%> (+7.27%) 1.00 <0.00> (ø)
src/main/java/seedu/address/ui/CommandBox.java 0.00% <ø> (ø) 0.00 <0.00> (ø)
src/main/java/seedu/address/ui/ItemCard.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
src/main/java/seedu/address/ui/ItemListPanel.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
... and 32 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6389d85...5187a4d. Read the comment docs.

chrystalquek added a commit to chrystalquek/tp that referenced this pull request Sep 28, 2020
…ucture-command

Shift Person into relevant packages
@codecov-io
Copy link

codecov-io commented Oct 11, 2020

Codecov Report

Merging #19 into master will decrease coverage by 6.42%.
The diff coverage is 65.26%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #19      +/-   ##
============================================
- Coverage     72.31%   65.88%   -6.43%     
- Complexity      400      430      +30     
============================================
  Files            70       80      +10     
  Lines          1228     1501     +273     
  Branches        124      159      +35     
============================================
+ Hits            888      989     +101     
- Misses          308      467     +159     
- Partials         32       45      +13     
Impacted Files Coverage Δ Complexity Δ
src/main/java/seedu/address/MainApp.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...n/java/seedu/address/commons/core/GuiSettings.java 69.23% <ø> (ø) 5.00 <0.00> (ø)
...in/java/seedu/address/commons/core/LogsCenter.java 78.37% <ø> (ø) 10.00 <0.00> (ø)
...ain/java/seedu/address/logic/commands/Command.java 100.00% <ø> (ø) 1.00 <0.00> (ø)
...va/seedu/address/logic/commands/RemoveCommand.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
...c/main/java/seedu/address/logic/parser/Prefix.java 91.66% <ø> (ø) 7.00 <0.00> (ø)
...eedu/address/logic/parser/RemoveCommandParser.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
...in/java/seedu/address/model/delivery/Delivery.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
.../java/seedu/address/model/util/SampleDataUtil.java 27.27% <0.00%> (+7.27%) 1.00 <0.00> (ø)
src/main/java/seedu/address/ui/CommandBox.java 0.00% <ø> (ø) 0.00 <0.00> (ø)
... and 72 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6389d85...33e30fd. Read the comment docs.

Wincenttjoi and others added 27 commits October 25, 2020 14:23
Update DG Implementation Add and CommandHistoryTraversal
Co-authored-by: Wincent Tjoi <46394439+Wincenttjoi@users.noreply.github.com>
Co-authored-by: Wincent Tjoi <46394439+Wincenttjoi@users.noreply.github.com>
Add description to CommandHistoryActivityDiagram

Split add manual test cases to existing/non-existing
…d-to-take-more-field

fix find-command bug by allowing more than 2 field
Change Ui.png, and fix UG summary table
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.

None yet

7 participants