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 PPP #230

Merged
merged 1 commit into from
Apr 14, 2024
Merged

Update PPP #230

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
30 changes: 22 additions & 8 deletions docs/team/eugenechanjiajun.md
View file
Open in desktop
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))
Loading