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-T13-3] FitEgo #69

Open
wants to merge 1,094 commits into
base: master
Choose a base branch
from

Conversation

benclmnt
Copy link

@benclmnt benclmnt commented Sep 14, 2020

FitEgo helps fitness instructors keep track of his/her customers easily, via CLI as he’s a fast typer so that he can spend more time on his clients / his routine rather than manually using alternative software like Excel to track administrative matters.

@benclmnt benclmnt changed the title [CS2103T-T13-3] [CS2103T-T13-3] FitEgo Sep 14, 2020
@codecov-commenter
Copy link

codecov-commenter commented Oct 4, 2020

Codecov Report

Merging #69 into master will decrease coverage by 6.51%.
The diff coverage is 60.24%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #69      +/-   ##
============================================
- Coverage     72.31%   65.80%   -6.52%     
- Complexity      400      501     +101     
============================================
  Files            70       89      +19     
  Lines          1228     1734     +506     
  Branches        124      183      +59     
============================================
+ Hits            888     1141     +253     
- Misses          308      525     +217     
- Partials         32       68      +36     
Impacted Files Coverage Δ Complexity Δ
src/main/java/seedu/address/MainApp.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...main/java/seedu/address/commons/core/Messages.java 0.00% <ø> (ø) 0.00 <0.00> (ø)
...ss/logic/commands/exceptions/CommandException.java 100.00% <ø> (ø) 2.00 <0.00> (ø)
...in/java/seedu/address/logic/parser/ParserUtil.java 97.22% <ø> (ø) 14.00 <0.00> (ø)
...a/seedu/address/logic/parser/client/CliSyntax.java 83.33% <ø> (ø) 1.00 <0.00> (?)
...ogic/parser/schedule/AddScheduleCommandParser.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
.../main/java/seedu/address/model/client/Address.java 80.00% <ø> (ø) 6.00 <0.00> (?)
...rc/main/java/seedu/address/model/client/Email.java 80.00% <ø> (ø) 6.00 <0.00> (?)
src/main/java/seedu/address/model/client/Name.java 80.00% <ø> (ø) 6.00 <0.00> (?)
...rc/main/java/seedu/address/model/client/Phone.java 80.00% <ø> (ø) 6.00 <0.00> (?)
... and 78 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...4d98628. Read the comment docs.

@codecov-io
Copy link

codecov-io commented Oct 10, 2020

Codecov Report

Merging #69 (cf2c26a) into master (b3e3b78) will increase coverage by 8.90%.
The diff coverage is 81.23%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #69      +/-   ##
============================================
+ Coverage     72.31%   81.22%   +8.90%     
- Complexity      400      780     +380     
============================================
  Files            70       95      +25     
  Lines          1228     2162     +934     
  Branches        124      262     +138     
============================================
+ Hits            888     1756     +868     
+ Misses          308      297      -11     
- Partials         32      109      +77     
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> (ø)
...ss/logic/commands/exceptions/CommandException.java 100.00% <ø> (ø) 2.00 <0.00> (ø)
...in/java/seedu/address/logic/parser/ParserUtil.java 97.22% <ø> (ø) 14.00 <0.00> (ø)
.../main/java/seedu/address/model/client/Address.java 80.00% <ø> (ø) 6.00 <0.00> (?)
...rc/main/java/seedu/address/model/client/Email.java 80.00% <ø> (ø) 6.00 <0.00> (?)
src/main/java/seedu/address/model/client/Name.java 72.72% <0.00%> (ø) 6.00 <0.00> (?)
...rc/main/java/seedu/address/model/client/Phone.java 80.00% <ø> (ø) 6.00 <0.00> (?)
src/main/java/seedu/address/model/tag/Tag.java 80.00% <ø> (ø) 5.00 <0.00> (ø)
... and 109 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 b3e3b78...cf2c26a. Read the comment docs.

GeNiaaz pushed a commit to GeNiaaz/tp that referenced this pull request Oct 11, 2020
chunyongg pushed a commit to chunyongg/Serenity that referenced this pull request Oct 12, 2020
BILLXYR pushed a commit to BILLXYR/tp that referenced this pull request Oct 13, 2020
…in-minimal-working

Implement basic functionalities
@benclmnt benclmnt force-pushed the master branch 3 times, most recently from a66949e to 0101b4e Compare November 6, 2020 02:12
kelvinvin and others added 30 commits November 9, 2020 18:27
* pb

* add page break

* add page break 3

* add page break 4

* add page break 4

* Avoid images breaking

* Remove page break

* Fix line break for 3.4

* Replace with page-break-before for some

* Add line breaks

* Add line breaks

* add line break

* remove line breaks

* try reducing space before toc

* add line break

* add line break

* add line break

* separate markdown and html

* Continue numbering after image

* Add line break after setting to A4

* add page breaks

* Swap ui orientation items

* Prevent image from page breaking

* Resize seq diagram

* Add page breaks

* add page breaks and hours spent

* add page break to design

* Add destroy marker for ViewSession in DG

* Update PPP

* Fix broken html tag

* use specific reposense link
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