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-W11-3] My Fitness Buddy #65

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

Conversation

junhui-phoon
Copy link

My Fitness Buddy helps Full-Time National Servicemen with PES ‘BP’ Status to track their overall change in fitness level by managing their daily calorie intake/output and weight changes during Basic Military Training.

@codecov-commenter
Copy link

Codecov Report

Merging #65 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #65   +/-   ##
=========================================
  Coverage     72.31%   72.31%           
  Complexity      400      400           
=========================================
  Files            70       70           
  Lines          1228     1228           
  Branches        124      124           
=========================================
  Hits            888      888           
  Misses          308      308           
  Partials         32       32           

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...354f7a2. Read the comment docs.

@codecov-io
Copy link

codecov-io commented Oct 8, 2020

Codecov Report

Merging #65 (82de4bd) into master (6389d85) will decrease coverage by 8.83%.
The diff coverage is 63.70%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #65      +/-   ##
============================================
- Coverage     72.31%   63.47%   -8.84%     
- Complexity      400      786     +386     
============================================
  Files            70      119      +49     
  Lines          1228     3020    +1792     
  Branches        124      348     +224     
============================================
+ Hits            888     1917    +1029     
- Misses          308      993     +685     
- Partials         32      110      +78     
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> (ø)
...main/java/seedu/address/commons/core/Messages.java 0.00% <ø> (ø) 0.00 <0.00> (ø)
...java/seedu/address/logic/commands/ExitCommand.java 100.00% <ø> (ø) 2.00 <0.00> (ø)
...java/seedu/address/logic/commands/ListCommand.java 100.00% <ø> (ø) 2.00 <0.00> (ø)
...ss/logic/commands/exceptions/CommandException.java 50.00% <ø> (-50.00%) 1.00 <0.00> (-1.00)
...ss/model/person/NameContainsKeywordsPredicate.java 62.50% <0.00%> (-37.50%) 5.00 <0.00> (-2.00)
.../java/seedu/address/model/util/SampleDataUtil.java 0.00% <0.00%> (-20.00%) 0.00 <0.00> (-1.00)
...c/main/java/seedu/address/ui/CalorieInputCard.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
...n/java/seedu/address/ui/CalorieInputListPanel.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
... and 150 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...82de4bd. Read the comment docs.

chunyongg added a commit to chunyongg/Serenity that referenced this pull request Oct 12, 2020
Wayne and others added 27 commits November 6, 2020 15:49
* Fixed a bug where null pointer exception is thrown in ProfileCardPanel after executing clear command
when there is no profile login
* Fixed the profile panels label duplication where each update or clear execution will instantiate new Profile Panels objects
by rewriting the UI update logic of the profile panels when clear/update operations are executed.
* Fixed the divider for profile panels so that users cannot adjust.
* Fixed FXML filename typo that caused Jar file to fail
* branch-BugFixes: (72 commits)
  Fix FXML filename typo in CommanderBmiStatsWindow
  Fix divider for profile panels
  Fix profile panels label duplication
  Fix null pointer exception after clear command
  Update ppp
  Edit PPP links in AboutUs
  Add PPP draft
  update ppp
  Update ppp
  PPP draft
  no message
  isSamePerson checks for ID only
  Ug typo fix for create profile
  Revert "Remove tags from storage and Day"
  Removed tags
  Remove tags from storage and Day
  Remove print statement
  Bug fix
  Update dg
  Fix PieChart not updated when new profile is created
  ...
* branch-BugFixes: (183 commits)
  Fix FXML filename typo in CommanderBmiStatsWindow
  Fix divider for profile panels
  Fix profile panels label duplication
  Fix null pointer exception after clear command
  Update ppp
  Edit PPP links in AboutUs
  Add PPP draft
  update ppp
  Update ppp
  PPP draft
  no message
  isSamePerson checks for ID only
  Ug typo fix for create profile
  Revert "Remove tags from storage and Day"
  Removed tags
  Remove tags from storage and Day
  Remove print statement
  Bug fix
  Update dg
  Fix PieChart not updated when new profile is created
  ...
* Added more test cases to test commander stats
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