-
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
Add a statistics box #103
Add a statistics box #103
Conversation
…t. Only need to fix UI now.
Checkstyle.
Codecov Report
@@ Coverage Diff @@
## master #103 +/- ##
============================================
+ Coverage 75.83% 76.19% +0.35%
- Complexity 695 726 +31
============================================
Files 101 103 +2
Lines 2032 2096 +64
Branches 236 244 +8
============================================
+ Hits 1541 1597 +56
+ Misses 433 426 -7
- Partials 58 73 +15
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.
Other than the comments mentioned, the rest LGTM!! 👍
src/test/java/seedu/address/model/AppointmentStatisticsTest.java
Outdated
Show resolved
Hide resolved
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.
LGTM! 👍
Now you can view the number of done/ upcoming appointments for today/this week in the UI.