-
Notifications
You must be signed in to change notification settings - Fork 5
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
Remove calorie command #121
Remove calorie command #121
Conversation
* master: Edit error message for calorie command Make edit accordingly to the review, and fix calorie command giving wrong error messages Add calorie command to be able to specify which date to add to Add CalorieManager Test Add CalorieManager
* master: Fix missing heading for UG Update storage and model DG Edit introduction of the DG Edit the logic component in DG Change intakes to inputs Update Ug and Dg Update ui diagrams Add stats functionality and other minor edits to the DG Fix checkstyle errors Add stats functionality to UG
* master: resolve merge conflicts Sort CI fix Sort Day and CalorieManager
Codecov Report
@@ Coverage Diff @@
## master #121 +/- ##
============================================
- Coverage 56.76% 55.63% -1.14%
- Complexity 467 476 +9
============================================
Files 93 95 +2
Lines 1811 1882 +71
Branches 191 202 +11
============================================
+ Hits 1028 1047 +19
- Misses 700 748 +48
- Partials 83 87 +4
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add missing Javadocs header to the classes under model/calorie package
import seedu.address.model.calorie.CalorieManager; | ||
import seedu.address.model.day.Day; | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Javadocs header
import seedu.address.model.calorie.Food; | ||
import seedu.address.model.calorie.Input; | ||
import seedu.address.model.calorie.Output; | ||
import seedu.address.model.calorie.Time; | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Javadocs header
import seedu.address.commons.core.index.Index; | ||
import seedu.address.logic.commands.RemoveCommand; | ||
import seedu.address.logic.parser.exceptions.ParseException; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Javadocs header
|
||
import java.util.ArrayList; | ||
import java.util.List; | ||
import java.util.logging.Logger; | ||
|
||
import seedu.address.commons.core.LogsCenter; | ||
import seedu.address.commons.core.index.Index; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Javadocs header
* master: Update version release number Fix checkstyle errors Change to BarChart for CalorieStats Add assertions to these classes removed additional assertions statement add sorting to daylist Enable assertions Fix checkstyle errors Add Dark Theme to Line Charts Fix bug where line charts are not updated Fix compilation errors caused by new CalorieManager Add functionality to show Calorie Stats Add functionality to show Weight Stats
* master: Fix bug where calorie data reset to 0 after editing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me
remove a certain calorie from a particular day
example: remove tp/out d/2020-10-23 i/2
removes a calorie output from day:2020-10-23 with the index of 2