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

Branch fix bugs #178

Merged
merged 6 commits into from
Nov 13, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions docs/DeveloperGuide.md
Original file line number Diff line number Diff line change
Expand Up @@ -152,7 +152,7 @@ A `Person` object encapsulates various attributes:
- `Age`: Patient's age
- `BloodType`: Patient's blood type
- `Set<Allergy>`: Patient's allergies
- `UniqueRecordList`: Patient's records of past visits to the clinic.
- `UniqueRecordList`: Patient's records from past visits to the clinic.
- `UniqueAppointmentList`: Patient's scheduled appointments.
- `isPinned`: Patient's pin status.

Expand Down Expand Up @@ -261,7 +261,7 @@ The `search` command filters the list of patients using one or more keywords.

1. **Parse User Input**: `FindCommandParser` checks for existence of the keyword(s) and creates an array of keywords.
2. **Create Predicate Object**: A `NameContainsKeywordsPredicate` object is instantiated during `FindCommandParser#parse(String)` and passed over to the `FindCommand`.
3. **Execute Command**: `Findommand#execute(Model)` finds patients containing keywords using `NameContainsKeywordsPredicate#test(Record)` and updates `FilteredPersonList`.
3. **Execute Command**: `FindCommand#execute(Model)` finds patients containing keywords using `NameContainsKeywordsPredicate#test(Record)` and updates `FilteredPersonList`.

<puml src="diagrams/SearchSequenceDiagram.puml"/>

Expand Down
4 changes: 2 additions & 2 deletions docs/UserGuide.md
Original file line number Diff line number Diff line change
Expand Up @@ -310,7 +310,7 @@ Example:

`addpatient n/John Doe i/T1234567A e/[email protected] p/12345678 g/M a/26 bt/AB+ al/Penicillin`
Adds a **Male** patient named **John Doe** whose NRIC is **T1234567A** , **26** years old, has **AB+** Blood Type, and is allergic to **Penicillin**.
His email and phone number is **[email protected]** and **12345678**, respectively.
His email and phone number is **[email protected]** and **12345678** respectively.

Screenshots:

Expand Down Expand Up @@ -428,7 +428,7 @@ Examples:

- `editpatient 2 p/92345678` Edits the `PHONE` of the patient with the `PATIENTID` of **2** to **92345678**. <br>

- `editpatient 3 al/aspirin` Edits the `ALLERGY` of the patient with the `PATIENTID` of **3** to**aspirin**. <br>
- `editpatient 3 al/aspirin` Edits the `ALLERGY` of the patient with the `PATIENTID` of **3** to **aspirin**. <br>

- `editpatient 2 p/92345678 al/aspirin` Edits the `PHONE` and `ALLERGY` of the patient with the `PATIENTID` of **2** to **92345678** and **aspirin**, respectively. <br>

Expand Down
2 changes: 1 addition & 1 deletion docs/diagrams/AddPatientSequenceDiagram.puml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ deactivate AddressBookParser
LogicManager -> AddCommand : execute(model)
activate AddCommand

AddCommand -> Model : addPerson(toAdd)
AddCommand -> Model : addPerson(newPerson)
activate Model

Model --> AddCommand
Expand Down
2 changes: 1 addition & 1 deletion docs/diagrams/SearchSequenceDiagram.puml
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ create CommandResult
FindCommand -> CommandResult
activate CommandResult

CommandResult --> FindCommand : result
CommandResult --> FindCommand
deactivate CommandResult

FindCommand --> LogicManager : result
Expand Down
Binary file modified docs/images/ryanongwx.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file modified docs/images/screenshots/editPatient - after.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file modified docs/images/screenshots/viewAppointment.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
11 changes: 6 additions & 5 deletions docs/team/ryanongwx.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,16 +11,16 @@

Given below are my contributions to the project.

* **Enhancement 1**: Search Patient
* **Enhancement 1**: Search Patient v1.2

- Broaden the scope of search such that it covers all the necessary information stored within all patients. This is to ensure that a doctor could search for any information on a patient when required. This is especially important in situations where doctors would like to search for all patients which have a certain condition or have a certain allergy.

* **New Feature 1**: Add Attach File Functionality
* **New Feature 1**: Add Attach File Functionality v1.3
- Allows doctors to attach a medical report documennt to a record. This allows efficient management of medical documents and allows doctors to keep track of their records in an organised manner.
- Files would be stored in their local storage of the device. Upon attaching the file, they would be able to open the document whenever they require it.
- This is extremely useful for attaching files such as blood test results that is vital to be kept together with patient records.

* **New Feature 2**: Add Encryption/Decryption Functionality
* **New Feature 2**: Add Encryption/Decryption Functionality v1.3
- Encrypts data from the application using AES encryption such that the data in the addressbook.json file is encrypted.
- This is to ensure privacy and confidentiality for patient data.
- This functionality was later scrapped due to the group prject constraints
Expand All @@ -35,12 +35,13 @@ Given below are my contributions to the project.
* **Project management**:
* Managed releases `v1.3` - `v1.5rc` (3 releases) on GitHub
* Filmed and stitched the video demo together
* Assessed testers' issues from PE-D

* **Documentation**:
* User Guide:
* Added documentation for the all features that I had implemented
- Added documentation for the all features that I had implemented
* Developer Guide:
* Added implementation details for the all features that I had implemented
- Added implementation details for the all features that I had implemented

* **Community**:
* PRs reviewed : [\#28](https://github.com/AY2324S1-CS2103T-T12-4/tp/pull/28), [\#62](https://github.com/AY2324S1-CS2103T-T12-4/tp/pull/62), [\152](https://github.com/AY2324S1-CS2103T-T12-4/tp/pull/152), [\#56](https://github.com/AY2324S1-CS2103T-T12-4/tp/pull/56)
Expand Down
Loading