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

Jump to bottom

Update Amy's PPP #189

Merged
merged 1 commit into from
Nov 11, 2023
Merged

Update Amy's PPP #189

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
46 changes: 19 additions & 27 deletions docs/team/ylyma.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
---
title: John Doe's Project Portfolio Page
title: Amy's Project Portfolio Page
---

### Project: TAfinder
Expand All @@ -8,38 +8,30 @@ TAfinder is a desktop resource management application used by NUS SOC professors

Given below are my contributions to the project.

* **New Feature**: Added the ability to undo/redo previous commands.
* What it does: allows the user to undo all previous commands one at a time. Preceding undo commands can be reversed by using the redo command.
* Justification: This feature improves the product significantly because a user can make mistakes in commands and the app should provide a convenient way to rectify them.
* Highlights: This enhancement affects existing commands and commands to be added in future. It required an in-depth analysis of design alternatives. The implementation too was challenging as it required changes to existing commands.
* Credits: *{mention here if you reused any code/ideas from elsewhere or if a third-party library is heavily used in the feature so that a reader can make a more accurate judgement of how much effort went into the feature}*
* **Code contributed**: [RepoSense link](https://nus-cs2103-ay2324s1.github.io/tp-dashboard/?search=ylyma&sort=groupTitle&sortWithin=title&timeframe=commit&mergegroup=&groupSelect=groupByAuthors&breakdown=true&checkedFileTypes=docs~functional-code~test-code&since=2023-09-22&tabOpen=true&tabType=authorship&tabAuthor=ylyma&tabRepo=AY2324S1-CS2103T-W10-1%2Ftp%5Bmaster%5D&authorshipIsMergeGroup=false&authorshipFileTypes=docs~functional-code~test-code&authorshipIsBinaryFileTypeChecked=false&authorshipIsIgnoredFilesChecked=false)

* **New Feature**: Added a history command that allows the user to navigate to previous commands using up/down keys.
* **New Feature**: Added the 'hide', 'unhide' and 'unhide-all' commands to the app.
* What it does: Allows the user to hide and unhide applicants from the list of applicants in the app.
* Justification: This feature is useful for the user to hide applicants that are not suitable for the job, and unhide them when they are suitable, helping to declutter the list of applicants.
* Highlights: This feature required a modification of several models to include new fields. I had to take care to preserve the immutability of the classes while doing so. I also had to make sure this command is applied to all future lists.

* **Code contributed**: [RepoSense link]()
* **New Feature**: Added a 'sort' command
* What it does: Allows the user to sort applicants by a specified field.
* Justification: This feature allows the most suitable applicants to be listed in order based on the user's criteria, helping them to make a decision more efficiently.
* Highlights: To allow flexibility of command when more fields are added in the future, I had to make sure the class is easily extendable to include new fields, while ensuring I stuck to the OOP principles. I also had to deliberate on how each field should be sorted.

* **Project management**:
* Managed releases `v1.3` - `v1.5rc` (3 releases) on GitHub

* **Enhancements to existing features**:
* Updated the GUI color scheme (Pull requests [\#33](), [\#34]())
* Wrote additional tests for existing features to increase coverage from 88% to 92% (Pull requests [\#36](), [\#38]())
* **Enhancement to existing feature**: Enhanced 'list' command
* What it does: Allow the 'list' command to filter applicants based on a specified field, in addition to listing all applicants.
* Justification: This feature allows the user to see a filtered list of applicants based on what they are looking for (e.g. bookmarked applicants), making their search for the right applicant more organised.
* Highlights: I had to make this feature easily extendable to new fields as well. I also had to modify some existing classes.

* **Documentation**:
* User Guide:
* Added documentation for the features `delete` and `find` [\#72]()
* Did cosmetic tweaks to existing documentation of features `clear`, `exit`: [\#74]()
* Added documentation for the features `hide`, `unhide`, `unhide-all`, `sort` and `list`
* Added the corresponding screenshots for each command
* Developer Guide:
* Added implementation details of the `delete` feature.

* **Community**:
* PRs reviewed (with non-trivial review comments): [\#12](), [\#32](), [\#19](), [\#42]()
* Contributed to forum discussions (examples: [1](), [2](), [3](), [4]())
* Reported bugs and suggestions for other teams in the class (examples: [1](), [2](), [3]())
* Some parts of the history feature I added was adopted by several other class mates ([1](), [2]())
* Added the hide command section under implementation

* **Tools**:
* Integrated a third party library (Natty) to the project ([\#42]())
* Integrated a new Github plugin (CircleCI) to the team repo
* **Contributions to team-based tasks**:
* Reviewed PRs for team members

* _{you can add/remove categories in the list above}_
Loading