Skip to content

Commit

Permalink
Merge pull request #223 from yytan25/update-ppp
Browse files Browse the repository at this point in the history
Update Yang Yi's PPP
  • Loading branch information
tanyyyming authored Nov 13, 2023
2 parents 89224fc + cba5b40 commit d4bcc4b
Showing 1 changed file with 42 additions and 17 deletions.
59 changes: 42 additions & 17 deletions docs/team/yytan25.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,33 +4,58 @@ title: Tan Yang Yi's Project Portfolio Page
---

### Project: BandConnect++
BandConnect++ helps musicians manage contact with other musicians to easily form a band of their preference. It is optimised for CLI users so that frequent tasks can be done faster by typing in commands.
BandConnect++ is a desktop application that helps music producers seeking talented musicians to manage their musician contacts and form a band of their preferences.
It is optimised for CLI users so that frequent tasks can be done faster by typing in commands.

Given below are my contributions to the project.

* **New Feature**: to be added soon
* What it does:
* Justification:
* Highlights:
* Credits:
* **New Feature 1: Delete a band** [\#89](https://github.com/AY2324S1-CS2103T-W11-3/tp/pull/89)
* What it does: Allows users to delete a band using the index of the band.
* Justification: It is a common use case for music producers to remove bands that are disbanded or otherwise no longer needed.
* Highlights: This enhancement involves updating the `FilteredBandList` to remove the band that is deleted.
In addition, the GUI will update the band list to reflect the removal of the band.
* Credits: I took inspiration from the `DeleteCommand` of the original AB3 and applied the same logic to the `FilteredBandList`.


* **New Feature 2: Remove a musician from a band** [\#93](https://github.com/AY2324S1-CS2103T-W11-3/tp/pull/93), [\#130](https://github.com/AY2324S1-CS2103T-W11-3/tp/pull/130)
* What it does: Allows users to remove a musician from a band using the indices of the musician and the band.
* Justification: It is a common use case for music producers to manage their bands and delete musicians who are no longer present.
* Highlights: This enhancement involves a new operation in the `UniqueBandList` to access the target `Band` and `Musician` objects,
and then removing the musician from the band. A new method, `updateFilteredBandMusicianList`, was also implemented to reflect the
updated band members in the GUI.
* Credits: I took reference from the implementation of `AddMusicianFromBandCommand` and applied the logic in reverse for this feature.


* **Code contributed**: [RepoSense link](https://nus-cs2103-ay2324s1.github.io/tp-dashboard/?search=yytan25&breakdown=false&sort=groupTitle%20dsc&sortWithin=title&since=2023-09-22&timeframe=commit&mergegroup=&groupSelect=groupByRepos)

* **Project management**:
* to be added soon

* **Enhancements to existing features**:
* to be added soon
* Organised the Commands and corresponding Parsers to sub-packages `band`, `musician` and `general` [\#97](https://github.com/AY2324S1-CS2103T-W11-3/tp/pull/97)


* **Team tasks contributions**:
* Reviewed PRs of team members and gave constructive feedback
* Made significant enhancements to the product to include the `Band` entity, which involved:
1. Implementing the `Band`, `UniqueBandList` and `BandName` classes [\#77](https://github.com/AY2324S1-CS2103T-W11-3/tp/pull/77)
2. Implementing all test classes in `seedu.address.model.band` [\#95](https://github.com/AY2324S1-CS2103T-W11-3/tp/pull/95)
3. Implementing the initial version of `FindBandCommand` [\#80](https://github.com/AY2324S1-CS2103T-W11-3/tp/pull/80)
4. General enhancements in `ModelManager` and `AddressBook`
* Did regular inspection of the `Band`-related sections of the code and ensured consistency across all methods [\#98](https://github.com/AY2324S1-CS2103T-W11-3/tp/pull/98)
* Enhanced the UG to cater to the target audience of the product


* **Project management**:
* Reviewed the code and made GitHub releases for the `v1.1`, `v1.2` and `v1.3` milestones.
* Wrapped up the `v1.1`, `v1.2` and `v1.3` milestones.
* Ensured that all issues were accounted for and managed the issue tracker for each milestone.


* **Documentation**:
* User Guide:
* to be added soon
* Developer Guide:
* to be added soon
* User Guide: [\#54](https://github.com/AY2324S1-CS2103T-W11-3/tp/pull/54) [\#150](https://github.com/AY2324S1-CS2103T-W11-3/tp/pull/150)
* Features: `DeleteBandCommand`, `RemoveMusicianFromBandCommand`
* Developer Guide: [\#117](https://github.com/AY2324S1-CS2103T-W11-3/tp/pull/117)
* Added user stories

* **Community**:
* to be added soon

* **Tools**:
* to be added soon
* **Community**:
* Participated in PE Dry Run and gave 7 detailed bug reports

0 comments on commit d4bcc4b

Please sign in to comment.