Skip to content

Commit

Permalink
Merge pull request #561 from Gabriel4357/PPP-final
Browse files Browse the repository at this point in the history
Reduce from 3 pages to 2 pages
  • Loading branch information
juliusgambe committed Nov 14, 2023
2 parents 0c0c29c + 6fa7faf commit 5d72132
Showing 1 changed file with 13 additions and 40 deletions.
53 changes: 13 additions & 40 deletions docs/team/gabriel4357.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,23 +5,16 @@ title: "Gabriel Seethor's Project Portfolio Page"

### Project: HomeBoss

#### Overview
HomeBoss is a desktop application used for managing deliveries for home business owners.
The user interacts with it using a CLI, and it has a GUI created with JavaFX.
It is written in Java, and has about 19 kLoC.

#### Summary of Contributions
Given below are my contributions to the project.

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

- ##### Enhancements Implemented
* Add a Delivery
* Update details of a Delivery
* Delete a Delivery
* Update details of a Customer
* Test Cases for all implemented features
Add a Delivery, Update details of a Delivery, Delete a Delivery, Update details of a Customer and Test Cases for all
implemented features

* **New Feature:** Add a Delivery
* What it does: This feature allows the user to create Deliveries by specifying the expected Delivery date,
Expand Down Expand Up @@ -63,21 +56,15 @@ Given below are my contributions to the project.
* Credits: The implementation of this feature was tweaked from the AB3 `edit` command.
* Related pull requests: [#111](https://github.com/AY2324S1-CS2103T-T13-3/tp/pull/111)


* Added JUnit tests for all features implemented by me. This included the creation of the necessary `Builder`
classes such as `DeliveryBuilder`, `DeliveryEditDescriptorBuilder`, `DeliveryAddDescriptorBuilder` and adding to
the `CommandTestUtil` to include valid and invalid parameters to different commands.

- ##### Contributions to the User Guide
* Structuring the UG
* Introduction
* Table of Contents
* Editing the data file
* Feature - Update details of a Customer
* Feature - View a list of Customers
* Feature - Add a Delivery
* Feature - Update details of a Delivery
* Feature - Update status of a Delivery
* Feature - Delete a Delivery
* Formatting and styling of the UG
* Ensuring consistency and correctness of the UG

* Structuring the UG, Introduction, Table of Contents, Editing the data file , Feature - Update details of a
Customer, Feature - View a list of Customers, Feature - Add a Delivery , Feature - Update details of a
Delivery, Feature - Update status of a Delivery, Feature - Delete a Delivery, Formatting and styling of the UG,
Ensuring consistency and correctness of the UG
Related pull requests:
[#38](https://github.com/AY2324S1-CS2103T-T13-3/tp/pull/38)
[#73](https://github.com/AY2324S1-CS2103T-T13-3/tp/pull/73)
Expand All @@ -88,17 +75,10 @@ Related pull requests:
[#416](https://github.com/AY2324S1-CS2103T-T13-3/tp/pull/416)
[#477](https://github.com/AY2324S1-CS2103T-T13-3/tp/pull/477)



- ##### Contributions to the Developer Guide
* Did 1/5 of the Use Cases
* Customer Edit Feature
* Customer Edit Activity Diagram
* Customer Edit Sequence Diagram
* Delivery Add Feature
* Delivery Add Activity Diagram
* Delivery Add Sequence Diagram
* Checked through the DG for correctness and made refinements/adjustments
Did 1/5 of the Use Cases, Customer Edit Feature, Customer Edit Activity Diagram , Customer Edit Sequence Diagram,
Delivery Add Feature, Delivery Add Activity Diagram, Delivery Add Sequence Diagram, Checked through the DG for
correctness and made refinements/adjustments

Related pull requests:
[#72](https://github.com/AY2324S1-CS2103T-T13-3/tp/pull/72)
Expand All @@ -109,9 +89,6 @@ Related pull requests:


- ##### Contributions to team-based tasks
* Added JUnit tests for all features implemented by me. This included the creation of the necessary `Builder`
classes such as `DeliveryBuilder`, `DeliveryEditDescriptorBuilder`, `DeliveryAddDescriptorBuilder` and adding to
the `CommandTestUtil` to include valid and invalid parameters to different commands.
* Added Logging and Assertions for the portion of my features
[#526](https://github.com/AY2324S1-CS2103T-T13-3/tp/pull/526)
* Maintained code quality for the portion of my features
Expand All @@ -120,13 +97,9 @@ Related pull requests:
* Fix various bugs along the way, some examples: [#321](https://github.com/AY2324S1-CS2103T-T13-3/tp/pull/321)
[#337](https://github.com/AY2324S1-CS2103T-T13-3/tp/pull/337)


- ##### Review/mentoring contribution
* Created issues, reviewed, approved and merged pull requests.
* Left comments on PRs to suggest improvements.
* Reviewed 32 PRs
[GitHub](https://github.com/AY2324S1-CS2103T-T13-3/tp/pulls?q=is%3Apr+is%3Aclosed+reviewed-by%3A%40me)


- ##### Contributions beyond the project team
* Reported bugs during PE dry-run

0 comments on commit 5d72132

Please sign in to comment.