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

Add MarkAppointmentAsDone functionalities #172

Merged
merged 2 commits into from
Oct 27, 2020

Conversation

porkeypine
Copy link

No description provided.

appointments (that are not missed) as done in appointment list
Change constructor of Appointment so that if the dateTime of appointment
is before current dateTime, it is automatically marked as DONE
@codecov-io
Copy link

Codecov Report

Merging #172 into master will increase coverage by 4.84%.
The diff coverage is 14.49%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #172      +/-   ##
============================================
+ Coverage     54.14%   58.98%   +4.84%     
- Complexity      473      555      +82     
============================================
  Files            99      103       +4     
  Lines          1932     2048     +116     
  Branches        207      229      +22     
============================================
+ Hits           1046     1208     +162     
+ Misses          821      759      -62     
- Partials         65       81      +16     
Impacted Files Coverage Δ Complexity Δ
src/main/java/team/baymax/MainApp.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...n/java/team/baymax/commons/core/time/DateTime.java 91.30% <0.00%> (-4.16%) 12.00 <0.00> (+1.00) ⬇️
...n/java/team/baymax/logic/commands/ExitCommand.java 100.00% <ø> (ø) 3.00 <0.00> (ø)
...mmands/appointment/MarkAppointmentDoneCommand.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
...ymax/logic/commands/patient/AddPatientCommand.java 92.85% <ø> (ø) 9.00 <0.00> (ø)
...x/logic/commands/patient/DeletePatientCommand.java 92.85% <ø> (ø) 8.00 <0.00> (ø)
...max/logic/commands/patient/FindPatientCommand.java 91.66% <ø> (ø) 7.00 <0.00> (ø)
...eam/baymax/logic/parser/AppointmentBookParser.java 66.66% <0.00%> (-6.07%) 12.00 <0.00> (ø)
...ointment/ListPatientAppointmentsCommandParser.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
.../appointment/MarkAppointmentDoneCommandParser.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
... and 28 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 d00b2df...bbba522. Read the comment docs.

@jianhandev jianhandev added this to the v1.3 milestone Oct 27, 2020
@jianhandev jianhandev added the priority.High Must do label Oct 27, 2020
@jianhandev jianhandev linked an issue Oct 27, 2020 that may be closed by this pull request
@jianhandev jianhandev merged commit 9392b9e into AY2021S1-CS2103T-W12-3:master Oct 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Appointment] Mark appointment as done
3 participants