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.

Sign up for GitHub

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 doc ppp #228

Merged
merged 3 commits into from
Nov 9, 2023
Merged

Branch doc ppp #228

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
871ba3b
Add contributions to features and Documentation
Kb-Tay Nov 5, 2023
f05f95e
Merge branch 'master' into branch-Doc-PPP
Kb-Tay Nov 8, 2023
f8a29db
Update PPP
Kb-Tay Nov 8, 2023
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
41 changes: 37 additions & 4 deletions docs/team/kb-tay.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,18 +5,51 @@ title: Aaron Tay's Project Portfolio Page

### Overview

To be added soon.
UNOFAS is a desktop app for **Financial Advisors (FA)** to manage client’s contacts, optimized for use via a **Command Line Interface (CLI)** while still having the benefits of a **Graphical User Interface (GUI)**. The app also includes features such as sorting, scheduling and other commands to query information quickly required by the FA.

### Summary of Contributions

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.
* 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.
* 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.
* 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.
* Pull Request [#133](https://github.com/AY2324S1-CS2103T-F12-1/tp/pull/133)


* **Code Contributed**: [RepoSense](https://nus-cs2103-ay2324s1.github.io/tp-dashboard/?search=kb-tay&breakdown=true)

* **Enhancements Implemented**: to be added soon.

* **Contributions to the UG**: to be added soon.
* **Enhancements Implemented**:
* Added testing for implemented features


* **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.
* Added documentation on implementation and design considerations for `complete` feature.

* **Contributions to the DG**: to be added soon.

* **Contributions to team-based tasks**: to be added soon.
* 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)