Merge branch 'master' into branch-generate-pdf

docs/AboutUs.md

@@ -17,18 +17,18 @@ You can reach us at the email `seer[at]comp.nus.edu.sg`
 [[github](http://github.com/techjay-c)]
 [[portfolio](team/techjay-c.md)]
 
-* Role: Project Advisor
+* Role: Developer
+* Responsibilities: Search, Filter, Delete Commands and Testing
 
 ### Freddy Chen You Ren
 
 <img src="images/freddychenyouren2.png" width="200px">
 
 [[github](https://github.com/freddychenyouren2)]
-
 [[portfolio](team/freddychenyouren2.md)]
 
-* Role: Team Lead
-* Responsibilities: Team Lead
+* Role: Developer
+* Responsibilities: Dentist Commands and Testing
 
 ### Qin Yan Er
 
@@ -38,7 +38,7 @@ You can reach us at the email `seer[at]comp.nus.edu.sg`
 [[portfolio](team/qyaner.md)]
 
 * Role: Developer
-* Responsibilities: Data
+* Responsibilities: Appointment Commands and Testing
 
 ### Ruth Lim Sze Ern
 
@@ -48,7 +48,7 @@ You can reach us at the email `seer[at]comp.nus.edu.sg`
 [[portfolio](team/ruth-lim.md)]
 
 * Role: Developer
-* Responsibilities: Dev Ops + Threading
+* Responsibilities: UI Enhancements, Calendar Commands and Testing
 
 ### Roald Ong
 
@@ -58,4 +58,4 @@ You can reach us at the email `seer[at]comp.nus.edu.sg`
 [[portfolio](team/synapseprogramming.md)]
 
 * Role: Developer
-* Responsibilities: UI
+* Responsibilities: Patient, Treatment Commands and Testing

docs/diagrams/AddDentistSequenceDiagram.puml

@@ -14,10 +14,10 @@ box Model MODEL_COLOR_T1
 participant ":Model" as Model MODEL_COLOR
 end box
 
-[-> LogicManager : execute("add-dentist n/John Doe p/98765432 e/[email protected] h/My Dental Clinics Serangoon Road S123456 s/Orthodontics y/5 t/Braces")
+[-> LogicManager : execute("add-dentist n/John Doe p/98765432 s/Orthodontics y/5")
 activate LogicManager
 
-LogicManager -> AddressBookParser : parseCommand("add-dentist n/John Doe p/98765432 e/[email protected] h/My Dental Clinics Serangoon Road S123456 s/Orthodontics y/5 t/Braces")
+LogicManager -> AddressBookParser : parseCommand("add-dentist n/John Doe p/98765432 s/Orthodontics y/5")
 activate AddressBookParser
 
 create AddDentistCommandParser
@@ -27,7 +27,7 @@ activate AddDentistCommandParser
 AddDentistCommandParser --> AddressBookParser
 deactivate AddDentistCommandParser
 
-AddressBookParser -> AddDentistCommandParser : parse("n/John Doe p/98765432 e/[email protected] h/My Dental Clinics Serangoon Road S123456 s/Orthodontics y/5 t/Braces")
+AddressBookParser -> AddDentistCommandParser : parse("n/John Doe p/98765432 s/Orthodontics y/5")
 activate AddDentistCommandParser
 
 create AddDentistCommand

docs/diagrams/LogicClassDiagram.puml

@@ -38,7 +38,7 @@ LogicManager --> Storage
 Storage --[hidden] Model
 Command .[hidden]up.> Storage
 Command .right.> Model
-note right of XYZCommand: XYZCommand = AddCommand, \nFindCommand, etc
+note right of XYZCommand: XYZCommand = AddDentistCommand, \nSearchDentistCommand, etc
 
 Logic ..> CommandResult
 LogicManager .down.> CommandResult

docs/team/techjay-c.md

@@ -11,7 +11,7 @@ including patient and dentist management.
 
 Given below are my contributions to the project.
 
-* **New Feature: Search Dentist**
+* **New Feature**: Search Dentist [\#59](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/59), [\#75](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/75)
     * **What it does:** Adds the ability to search for dentists in ToothTracker based on either their `DENTIST_ID` or keywords in their names.
     * **Justification:** This feature enhances the user experience by providing quick access to specific dentists and facilitating efficient information retrieval.
     * **Credits:**
@@ -20,30 +20,30 @@ Given below are my contributions to the project.
         - Third-party libraries: N/A
 
 
-* **New Feature: Search Patient**
+* **New Feature**: Search Patient [\#60](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/60), [\#78](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/78)
     * **What it does:** Adds the ability to search for patients in ToothTracker based on either their `PATIENT_ID` or keywords in their names.
     * **Justification:** This feature enhances the user experience by providing quick access to specific patients and facilitating efficient information retrieval.
     * **Credits:**
         - Third-party libraries: N/A
 
 
-* **New Feature: Delete Patient**
+* **New Feature**: Delete Patient [\#68](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/68)
     * **What it does:** Introduces the ability to delete a patient from ToothTracker using their `PATIENT_ID`.
     * **Justification:** Enables users to manage their patient database effectively by allowing patient records to be removed easily.
     * **Credits:**
         - No code/ideas reused.
         - Third-party libraries: N/A
 
 
-* **New Feature: Filter Dentist**
+* **New Feature**: Filter Dentist [\#110](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/110)
     * **What it does:** Adds the functionality to filter dentists based on specified attributes such as their name, phone, specialization, experience, email, address, and tags.
     * **Justification:** Enhances the search capabilities, allowing users to narrow down dentist results based on a set of specified criteria.
     * **Credits:**
         - No code/ideas reused.
         - Third-party libraries: N/A
 
 
-* **New Feature: Filter Patient**
+* **New Feature**: Filter Patient [\#104](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/104)
     * **What it does:** Adds the functionality to filter patients based on specified attributes such as their name, phone, birthday, gender, remark, treatment, email, address and tags.
     * **Justification:** Enhances the search capabilities, allowing users to narrow down patient results based on a set of specified criteria.
     * **Credits:**
@@ -55,42 +55,47 @@ Given below are my contributions to the project.
 
 
 * **Project management**:
-    * Managed releases `v1.1` - `v1.4` ( releases) on GitHub
+    * Oversaw the successful management of releases `v1.1` through `v1.4` on GitHub, ensuring seamless and well-documented deployment processes.
+    * Led the team in delivering on-time, meeting project milestones, and consistently achieving deliverables.
+    * Formulated and implemented effective workflows to maintain a high standard of work quality across the entire team, fostering a culture of excellence and collaboration.
+    * Actively engaged in strategic planning and decision-making, adapting project management strategies as needed to address evolving priorities and challenges.
+    * Implemented measures to enhance team productivity and efficiency, contributing to the overall success of the project.
 
 
 * **Contributions to team-based tasks**:
     * Enabled Assertions in team repository, enhancing code quality and reliability 
 
 
 * **Enhancements to existing features**:
-    * Add Search Dentist Feature [\#59](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/59)
-    * Add Search Patient Feature [\#60](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/60)
-    * Update Search Dentist Feature [\#75](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/75)
-    * Update Search Dentist Feature [\#78](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/78)
-    * Add Filter Patient Feature [\#104](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/104)
-    * Add Filter Dentist Feature [\#110](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/110)
-    * Update Filter Dentist and Filter Patient Feature [\#117](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/117)
-    * Add Unit Tests for Search Dentist and Search Patient [\#132](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/132)
-    * Add Unit Tests for Filter Dentist and Filter Patient [\#132](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/132)
-    * Add Unit Tests for Delete Dentist and Delete Patient [\#136](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/136)
-    * Update Filter Dentist and Filter Patient Feature [\#139](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/139)
-    * Add Unit Tests for attributes in Filter Dentist and Filter Patient Feature [\#209](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/209)
+    * Add Unit Tests for Search Dentist and Search Patient commands [\#132](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/132)
+    * Update Filter Dentist and Filter Patient commands to include improved validation for the attributes being filtered [\#132](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/132)
+    * Add Unit Tests for Filter Dentist and Filter Patient commands [\#132](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/132)
+    * Add Unit Tests for Delete Dentist and Delete Patient commands [\#136](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/136)
+    * Update Filter Dentist and Filter Patient commands to support case-insensitive filtering [\#139](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/139)
+    * Add Unit Tests for attributes in Filter Dentist and Filter Patient commands [\#209](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/209)
 
 
 * **Documentation**:
     * User Guide:
-        * Update Use Cases for Search Dentist, Delete Dentist [\#38](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/38)
+        * Update Use Cases for Search Dentist, Delete Dentist commands [\#38](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/38)
         * Update User Guide and Command Descriptions [\#114](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/114) 
         * Added documentation for the features `search-dentist`, `search-patient`, `delete-dentist`, `delete-patient`, `filter-dentist`, `filter-patient` [\#145](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/145)
-        * Add user guide updates #152 [\#152](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/152)
+        * Update the User Guide to ensure that the documented command formats are accurate and that the descriptions precisely reflect the corresponding functionalities, enforcing consistency throughout the guide [\#152](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/152)
+        * Conduct proof-reading of the User Guide for consistency and grammatical accuracy [\#233](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/233)
 
-  * Developer Guide:
 
-      * Add Activity and Sequence Diagram for Search Dentist [\#124](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/124)
-      * Add Activity and Sequence Diagram for Filter Dentist [\#134](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/134)
-      * Add Activity and Sequence Diagram for Add Dentist, Filter Patient, Filter Dentist [\#214](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/214)
-      * Update diagrams in DG(Search Dentist, Filter Dentist, Delete Dentist) [\#214](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/214)
+  * Developer Guide:
+      * Add Activity and Sequence Diagrams for Search Dentist commands [\#124](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/124)
+      * Add Activity and Sequence Diagrams for Filter Dentist commands [\#134](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/134)
+      * Add Activity and Sequence Diagrams for Add Dentist, Filter Patient, Filter Dentist commands [\#214](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/214)
+      * Add and Update Activity and Sequence Diagrams for Search Dentist, Filter Dentist, Delete Dentist commands [\#214](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/214)
+      * Add and Update Use Cases for Search and Filter Commands for Dentist and Patient [\#238](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/238)
+      * Conduct proof-reading of the Developer Guide for consistency and grammatical accuracy [\#238](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/238)
 
 
 * **Community**:
-    * PRs reviewed (with non-trivial review comments): [\#84](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/84), [\#143](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/143), [\#154](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/154), [\#]()
+    * PRs reviewed (with non-trivial review comments): [\#84](https://github.com/AY2324S1-CS2103T-W10-3/tp/pull/84)
+    * Reported bugs and suggestions for other teams in the class: ([Link to bug report repo](https://github.com/techjay-c/ped))
+    * Conducted intensive testing on application to discover more bugs 
+      * Name attribute for `add-patient` and `add-dentist` commands allows numbers to be added
+      * Mandatory fields for `add-patient` and `add-dentist` commands were not enforced