Skip to content

Commit

Permalink
Update the enhancement contributed
Browse files Browse the repository at this point in the history
  • Loading branch information
g13nd0n-admin committed Nov 12, 2024
1 parent 663ad03 commit ee6b995
Showing 1 changed file with 42 additions and 5 deletions.
47 changes: 42 additions & 5 deletions docs/team/g13nd0n.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,11 @@ appointments for the day upon login. The application is optimised for Command Li
efficient management of patient information and appointments.

### Summary of Contributions
- Code Contributed: [RepoSense Link](https://nus-cs2113-ay2425s1.github.io/tp-dashboard/?search=g13nd0n)
- Code Contributed: [RepoSense Link](https://nus-cs2113-ay2425s1.github.io/tp-dashboard/?search=g13nd0n&breakdown=true&sort=groupTitle%20dsc&sortWithin=title&since=2024-09-20&timeframe=commit&mergegroup=&groupSelect=groupByRepos&checkedFileTypes=docs~functional-code~test-code~other)


- Enhancements implemented:
New Features
New Features:
- Add appointment for future schedule
- What it does: allow user to add appointments made with patients
- Justifications: This features improve the product by allowing user to track their time to avoid possible
Expand All @@ -27,6 +29,8 @@ efficient management of patient information and appointments.
methods and sorting them out. Additionally, implementation will need to ensure that regardless the order of the
appointment is added, it will be sorted appropriately after the appointment has been added.
while comparison are correct
- There is a guard clause added to prevent users from inputting different names despite the nric inputted being
the same.
- Delete appointments
- What it does: delete appointments based on the specific nric, date and time given
- Justification: This feature improves product by allowing user to delete the appointments that might have been
Expand All @@ -39,8 +43,9 @@ efficient management of patient information and appointments.
- Find appointments based on specific keywords such as date, time, name or nric
- What it does: Display all appointments based on the search keyword
- Justifications: This feature improves the product by allowing users to find the appointments for the specific
name, nric, date or time instead of listing all appointments and look for the appointments wanted. The search
feature is case-insensitive.
name, nric, date or time instead of listing all appointments and look for the appointments wanted.
- Highlights: The search feature is case-insensitive. The searching of appointments do not require the name to be
- fully spelled. Partial match of the name will also be shown.
- Appointments Notice upon logging in to the system
- What it does: Display all the appointments for the day
- Justification: This feature improves the product by removing the need for the user to list all appointments and
Expand All @@ -52,28 +57,60 @@ efficient management of patient information and appointments.
- Highlights: The appointments that are removed would be those that are before the time that the user logged off.
Implementations would be required to check both date and time before removal and not simply removing all the
appointments for the day.
- Added guard clause to the `add` function to prevent users from inputting same details of the patient causing
multiple records of the same patient.
- Wrote testing for the v1.0 project. #75
- Wrote additional testing for the new features added. Included testing of methods defined in the `Appointment` and
`AppointmentRecord` classes. #248


- Contributions to User Guide:
- Added documentation for the features `delete`, `appointment`, `deleteAppointment`, `listAppointments` and
`findAppointment` #205


- Contributions to Developer Guide:
- Added class diagram for `Appointment` and `AppointmentRecord` class
- Added details, sequence diagram and object diagram for the Appointment Feature
- Added information for the Glossary
- Check Contributions to Developer Guide for UML diagrams


- Contributions to team-based task :
- Convert the extraction of patient's details into a method since it has been used repeatedly #343
- Remove all the magic number when extracting patient's details #343
- Fixing bugs identified by other students #345, #358, #366
- Maintain the issue tracker


- Review/mentoring contributions:
- Help teammates in solving JUnit test issues e.g. System.lineSeparator() was not replaced with \n
- Help fixed the CI issues that was accumulated over time #75
- Advised teammates on how to create guard clause to prevent future dates from being added when adding new visit
dates


- Contributions beyond the project team:
- Help to spot bugs during dry run practical
- Peer review given to certain groups regarding their product and Developer Guide

---
## Contributions to the Developer Guide
```
The appointment mechanism is facilitated by `CommandHandler`. It creates an appointment slot via the `Appointment`
class and records it within the `AppointmentRecord` class. The appointment records will then be saved into
`FileHandler`.
- `CommandHandler.appointment(String input, AppointmentRecord appointmentRecord)` - check the input to check if valid
- `FileHandler.initFile(AppointmentRecord appointmentRecord)` - loads any saved appointment records
- `appointmentRecord.checkAvailability(LocalDate date, LocalTime time)` - check for availability at given time and date
- `AppointmentRecord.addAppointment(Appointment appointment)` - add appointment to the appointment record
- `Appointment(String name, String nric, String date, String time)` - creates appointment with the given name, nric,
- date and time
- `FileHandler.autosave(AppointmentRecord appointmentRecord)` - save the appointment records
Doctor enters the command: `appointment n/Patricia Chan ic/S9870789B date/06-11-2024 time/14:30`
```
- Appointment Class Diagram\
![Appointment.png](..%2FAppointment.png)
- AppointmentRecord Class Diagram\
Expand All @@ -89,7 +126,7 @@ Appendix D : Glossary
- Mainstream OS: Windows, Linux, Unix, MacOS
- NRIC: National Registration Identity Card (Identification Number)
```

---
## Contributions to the User Guide
```
Deleting a Patient Record
Expand Down

0 comments on commit ee6b995

Please sign in to comment.