Skip to content

Commit

Permalink
Merge pull request #230 from EugeneChanJiajun/master
Browse files Browse the repository at this point in the history
Update PPP
  • Loading branch information
daryltay415 committed Apr 14, 2024
2 parents 83dd6ff + 4bfa388 commit 02c8480
Showing 1 changed file with 22 additions and 8 deletions.
30 changes: 22 additions & 8 deletions docs/team/eugenechanjiajun.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,22 +11,34 @@ The app offers search, filtering, and currency exchange features, ensuring a smo

Given below are my contributions to the project.

#### Added the ability to save existing list information into a text file.
* New Feature: Added the ability to save existing list information into a text file.
* What it does: Allows the user to save their existing travel activity plans into a text file so that it can be
reloaded later on.
* Justification: This feature improves the product significantly because after a user inputs a certain travel plan,
they would want to be able to come back and update the list anytime they want.
* Highlights: This feature needs to be updated everytime a new feature is added into the project.

#### Handled input parsing for the chatbot.
* New Feature: Handled input parsing for the chatbot.
* What it does: The parser class handles all the input made by the user into the chatbot an accurately parses the
command to execute the appropriate tasks.
* Justification: This feature ensures that all command features implemented by the team is handled properly and the
appropriate output is produced
* Highlights: This feature needs to be updated everytime a new feature is added into the project.

* System testings: Tested most features to ensure that all major bugs are squashed.
* What is done: Everytime a new feature is added, ensure that the feature works well with all other existing features
and any possible bugs and regressions that may happen.
* Justification: This project has multiple features that require existing features to work together. Therefore, when
a feature is added, changes need to be made on other features. This may lead to regressions which needs to be fixed.
* Highlights: System testing affects existing commands and features and requires in-depth analysis of design
alternatives. As there are multiple features that have a relation with one another. (EG. Save feature needs to be
updated to ensure that new features are saved)

#### Code contributed: [RepoSense link](https://nus-cs2113-ay2324s2.github.io/tp-dashboard/?search=eugene&sort=groupTitle&sortWithin=title&timeframe=commit&mergegroup=&groupSelect=groupByRepos&breakdown=true&checkedFileTypes=docs~functional-code~test-code~other&since=2024-02-23&tabOpen=true&tabType=authorship&tabAuthor=EugeneChanJiajun&tabRepo=AY2324S2-CS2113-T12-4%2Ftp%5Bmaster%5D&authorshipIsMergeGroup=false&authorshipFileTypes=docs~functional-code~test-code~other&authorshipIsBinaryFileTypeChecked=false&authorshipIsIgnoredFilesChecked=false)

#### Project Management
* Managed releases `v2.0` on GitHub
* Create milestones and issues to ensure consistent updates to the project

#### Enhancement to existing features
* Added additional tests for existing features to increase coverage (Pull request [#197]()https://github.com/AY2324S2-CS2113-T12-4/tp/pull/197 and [#225](https://github.com/AY2324S2-CS2113-T12-4/tp/pull/225))
Expand All @@ -35,12 +47,14 @@ Given below are my contributions to the project.

#### Documentation
* User Guide:
* Added documentation for how to run the application
* Added documentation for the features
* Added documentation for how to run the application (Pull request [#120](https://github.com/AY2324S2-CS2113-T12-4/tp/pull/120))
* Added documentation for the features (Pull request [#123](https://github.com/AY2324S2-CS2113-T12-4/tp/pull/123))

* Developer Guide:
* Added implementation details for the `save` feature.
* Added sequence diagram for the `save` feature.
* Added class diagram for the `save` feature.

* Added implementation details for the `save` feature (Pull request [#134](https://github.com/AY2324S2-CS2113-T12-4/tp/pull/134))
* Added sequence diagram for the `save` feature. (Pull request [#100](https://github.com/AY2324S2-CS2113-T12-4/tp/pull/100))
* Added class diagram for the `save` feature. (Pull request [#221](https://github.com/AY2324S2-CS2113-T12-4/tp/pull/221))
* Added user stories in the developer guide. (Pull request [#125](https://github.com/AY2324S2-CS2113-T12-4/tp/pull/125))

#### Community
* Reported bugs and suggestions for other teams in the class (example [1](https://github.com/nus-cs2113-AY2324S2/tp/pull/42/files), [2](https://github.com/EugeneChanJiajun/ped/issues))

0 comments on commit 02c8480

Please sign in to comment.