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

Modify iynixil PPP #273

Merged
merged 1 commit into from
Apr 15, 2024
+12 −21
Merged

Modify iynixil PPP #273

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
33 changes: 12 additions & 21 deletions docs/team/iynixil.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,38 +9,29 @@ StaffConnect offers convenience and peace of mind to a struggling student with m

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}*
* **New Feature**: Added the ability to filter persons in staff book.
* What it does: allows the user to filter for persons with certain values in specified attributes.
* Justification: This feature improves the product significantly because a user can efficiently view the staff they want to see that possess certain characteristics (e.g. They teach a certain module)
* 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 relatively simple as it refers to the existing `find` command feature.

* **New Feature**: Added a history command that allows the user to navigate to previous commands using up/down keys.

* **Code contributed**: [RepoSense link]()
* **Code contributed**: [RepoSense link](https://nus-cs2103-ay2324s2.github.io/tp-dashboard/?search=iynixil&breakdown=true&sort=groupTitle%20dsc&sortWithin=title&since=2024-02-23&timeframe=commit&mergegroup=&groupSelect=groupByRepos&checkedFileTypes=docs~functional-code~test-code~other)

* **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]())

* **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 `Filtering persons` [\#93](https://github.com/AY2324S2-CS2103-F08-3/tp/pull/93)
* Added warnings for the features `delete`, `meeting-delete`, `clear`, `refresh` [\#229](https://github.com/AY2324S2-CS2103-F08-3/tp/pull/229)
* Added explanation for attributes in attribute summary table `Attribute Summary` [\#238](https://github.com/AY2324S2-CS2103-F08-3/tp/pull/238)
* Developer Guide:
* Added implementation details of the `delete` feature.
* Added implementation details of the `filter` feature. [\#146](https://github.com/AY2324S2-CS2103-F08-3/tp/pull/146)
* Updated Appendix sections. [\#257](https://github.com/AY2324S2-CS2103-F08-3/tp/pull/257)

* **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]())

* **Tools**:
* Integrated a third party library (Natty) to the project ([\#42]())
* Integrated a new Github plugin (CircleCI) to the team repo
* PRs reviewed (with non-trivial review comments): [link to all PRs reviewed by iynixil](https://github.com/AY2324S2-CS2103-F08-3/tp/pulls?q=is%3Apr+reviewed-by%3Aiynixil)
* Reported bugs and suggestions for other teams in the class: [link to all bugs found by iynixil](https://github.com/AY2324S2-CS2103T-W08-1/tp/issues?q=is%3Aissue+ped%2Fiynixil)

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