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-T15-3] WardWatch #130

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

Conversation

clarud
Copy link

@clarud clarud commented Sep 21, 2024

WardWatch is an app designed for hospital doctors that streamlines patient management by providing essential information such as ward location, diagnosis, and medication dispensed. It enhances workflow efficiency, reduces the risk of missed visits, and supports better patient care.

@clarud clarud changed the title [CS2103T] WardWatch [CS2103T-T15-3] WardWatch Sep 21, 2024
Copy link

codecov bot commented Oct 1, 2024

Codecov Report

Attention: Patch coverage is 61.14865% with 115 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...rc/main/java/seedu/address/ui/AppointmentCard.java 0.00% 17 Missing ⚠️
...edu/address/logic/commands/DeleteNotesCommand.java 44.00% 11 Missing and 3 partials ⚠️
src/main/java/seedu/address/ui/MainWindow.java 0.00% 13 Missing ⚠️
...in/java/seedu/address/ui/AppointmentListPanel.java 0.00% 12 Missing ⚠️
...main/java/seedu/address/commons/util/DateUtil.java 42.85% 8 Missing ⚠️
...s/model/person/FieldContainsKeywordsPredicate.java 27.27% 8 Missing ⚠️
.../seedu/address/logic/parser/AddressBookParser.java 0.00% 6 Missing and 1 partial ⚠️
...c/main/java/seedu/address/model/person/Person.java 41.66% 5 Missing and 2 partials ⚠️
.../seedu/address/logic/parser/FindCommandParser.java 0.00% 4 Missing and 2 partials ⚠️
...du/address/logic/parser/AddNotesCommandParser.java 42.85% 2 Missing and 2 partials ⚠️
... and 10 more
Files with missing lines Coverage Δ Complexity Δ
...n/java/seedu/address/commons/core/GuiSettings.java 76.92% <ø> (ø) 8.00 <0.00> (ø)
src/main/java/seedu/address/logic/Messages.java 90.47% <100.00%> (+9.52%) 7.00 <0.00> (+4.00)
.../address/logic/commands/AddAppointmentCommand.java 75.75% <100.00%> (+75.75%) 6.00 <0.00> (+6.00)
.../java/seedu/address/logic/commands/AddCommand.java 100.00% <100.00%> (ø) 12.00 <0.00> (+4.00)
.../seedu/address/logic/commands/AddNotesCommand.java 100.00% <100.00%> (+40.00%) 11.00 <0.00> (+3.00)
...va/seedu/address/logic/commands/DeleteCommand.java 100.00% <ø> (ø) 7.00 <0.00> (ø)
...java/seedu/address/logic/commands/EditCommand.java 97.75% <100.00%> (+0.25%) 17.00 <0.00> (+4.00)
...java/seedu/address/logic/commands/FindCommand.java 100.00% <ø> (ø) 6.00 <0.00> (ø)
...java/seedu/address/logic/commands/ListCommand.java 100.00% <ø> (ø) 2.00 <0.00> (ø)
...du/address/logic/commands/ScheduleDateCommand.java 100.00% <100.00%> (ø) 6.00 <6.00> (?)
... and 35 more

... and 3 files with indirect coverage changes

niclammm and others added 28 commits October 23, 2024 11:54
Enhancement: make a list to keep track of appointments
Merge master into new-schedule-parser-branch to include recent updates
The test methods in the Predicate classes do not check if person is
null.

requiring the person to be non null adds an additional layer of
check to be done, making our product less susceptible to bugs.
Bug fix: Improve defensive programming measures
The primary functionality of the schedule feature is to show the
user's schedule for a particular day. Hence, it makes sense for
the appointment list to be sorted.

Doing so will improve user experience as the results will be sorted,
making it easier for them to track their activities.

Let's use a SortedList to wrap around the filteredList to achieve
this functionality.
choonzies and others added 30 commits November 7, 2024 17:39
Find command requires a complete word match. This makes the
command less flexible.

Find command cannot search via patient notes.

Let's implement find command to search via substring match, and
also such that it can search via patient notes.
Find command cannot search via appointment description.

Enabling this functionality will enhance user experience.
UG is not updated with the enhanced find features, and is
also missing some other key details.
Update UG to reflect new functionalities
Add Appointment Command Test for overlapping Dates
Add Jing Choon part of Effort under Appendix
Add add appointment sequence diagram
Update DG with table of content and other info
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.

5 participants