Skip to content

Commit

Permalink
Merge branch 'master' of https://github.com/AY2324S1-CS2103T-F12-1/tp
Browse files Browse the repository at this point in the history
…into branch-docs-final
  • Loading branch information
AlyssaPng committed Nov 13, 2023
2 parents d7e9c2b + 6276c9e commit bb6529a
Showing 1 changed file with 14 additions and 14 deletions.
28 changes: 14 additions & 14 deletions docs/team/kb-tay.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,22 +11,20 @@ UNOFAS is a desktop app for **Financial Advisors (FA)** to manage client’s con

Given below are my contributions to the project.

* **New Feature**: Added the ScheduleItem class as a new field to Person
* What it does: Ensures that a Person has a 1-to-1 relationship with a ScheduleItem object.
* Justification: This feature provides users the ability to associate appointments with their clients.
* Highlights: An abstract class is used to allow methods to be executed on the person's appointment field without needing to know if it is a null appointment or appointment.
* **New Feature**: Added the `ScheduleItem` class as a new field to Person
* Justification: This feature provides users the ability to associate appointments with their clients.
* Pull Request [#70](https://github.com/AY2324S1-CS2103T-F12-1/tp/pull/70)

<br>

* **New Feature** Added the ability to schedule appointments.
* What it does: Allows the user to create an appointment with the specified client.
* **New Feature**: Added the ability to `schedule` appointments.
* What it does: Allows the user to create an appointment associated with the specified client.
* Justification: This feature provides users the ability to schedule and keep track of appointments with their clients.
* Pull Request [#70](https://github.com/AY2324S1-CS2103T-F12-1/tp/pull/70)

<br>

* **New Feature** Added the ability to complete appointments.
* **New Feature**: Added the ability to complete appointments.
* What it does: Allows users to clear appointments completed from the contact book.
* Justification: With this feature, user can keep track of appointments completed.
* Highlights: This command allows for more flexibility, giving users the choice to clear appointments by user's index or by a date.
Expand All @@ -38,25 +36,27 @@ Given below are my contributions to the project.

* **Enhancements Implemented**:
* Added testing for implemented features
* Refactored `ScheduleItem` class to adhere to LSP. [#108](https://github.com/AY2324S1-CS2103T-F12-1/tp/pull/108) [#217](https://github.com/AY2324S1-CS2103T-F12-1/tp/issues/217)
* Refactored `CompleteCommand` to better adhere to the command pattern.
* Refactored `ScheduleItem` class to adhere to LSP. [#108](https://github.com/AY2324S1-CS2103T-F12-1/tp/pull/108)
* Refactored `CompleteCommand` to better adhere to the command pattern. [#217](https://github.com/AY2324S1-CS2103T-F12-1/tp/issues/217)


* **Contributions to the UG**:
* Added documentation for `schedule` command and `complete` under feature section. [#137](https://github.com/AY2324S1-CS2103T-F12-1/tp/pull/137) [#135](https://github.com/AY2324S1-CS2103T-F12-1/tp/pull/137) [#210](https://github.com/AY2324S1-CS2103T-F12-1/tp/pull/210)


* **Contributions to the DG**:
* Added class diagram and sequence diagram for `schedule` feature.
* Added documentation on implementation and design considerations for `schedule` feature.
* Added sequence and activity diagram for `complete` feature.
* Document the overview of how `Command` is executed.
* Added documentation, class diagram and sequence diagram for `schedule` feature.
* Added documentation on design considerations for `Appointment` field. [#137](https://github.com/AY2324S1-CS2103T-F12-1/tp/pull/137)
* Added sequence and activity diagram for `complete` feature. [#209](https://github.com/AY2324S1-CS2103T-F12-1/tp/pull/209/files)
* Added documentation on implementation and design considerations for `complete` feature.


* **Community**:
* Review PRs of teammates. [#109](https://github.com/AY2324S1-CS2103T-F12-1/tp/pull/109) [#205](https://github.com/AY2324S1-CS2103T-F12-1/tp/pull/205) [#110](https://github.com/AY2324S1-CS2103T-F12-1/tp/pull/110) [#256](https://github.com/AY2324S1-CS2103T-F12-1/tp/pull/256)
* Review DG and suggested bug fixes for teammates. [#285](https://github.com/AY2324S1-CS2103T-F12-1/tp/issues/285) [#278] (https://github.com/AY2324S1-CS2103T-F12-1/tp/issues/278)
* Review DG and suggested bug fixes for teammates. [#285](https://github.com/AY2324S1-CS2103T-F12-1/tp/issues/285) [#278](https://github.com/AY2324S1-CS2103T-F12-1/tp/issues/278)


* **Contributions to team-based tasks**
* Update Developer Guide - Update diagrams for the general components.
* Update Developer Guide - Update diagram for `Model` and `Storage`, checked for overall correctness. [#128](https://github.com/AY2324S1-CS2103T-F12-1/tp/pull/128)
* Handled PE-D bug triaging.

0 comments on commit bb6529a

Please sign in to comment.