From 677df8e64dbb7219ada595698ffb3010bc18d8be Mon Sep 17 00:00:00 2001 From: seox123 Date: Mon, 7 Nov 2022 23:12:02 +0800 Subject: [PATCH 1/4] Update project portfolio page --- docs/team/seox123.md | 52 +++++++++++++++---------- src/main/java/seedu/waddle/MainApp.java | 2 +- 2 files changed, 33 insertions(+), 21 deletions(-) diff --git a/docs/team/seox123.md b/docs/team/seox123.md index ea62a3885be..8b10da2e99d 100644 --- a/docs/team/seox123.md +++ b/docs/team/seox123.md @@ -9,37 +9,49 @@ Waddle is a simple, no-frills travel planning application catered to people who Given below are my contributions to the project. -* **New Feature**: Add an edit item command. +* **New Feature**: Add an edit item command. [#49](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/49) * What it does: This command allows users to edit an item in their unscheduled item list. * Justification: This feature improves the product significantly as users can easily edit minor details in the items in their item list without having to delete and re-add the entire item. - * Highlights: The - - -* **Code contributed**: [RepoSense link](https://nus-cs2103-ay2223s1.github.io/tp-dashboard/?search=ciaoosuuu&breakdown=true&sort=groupTitle&sortWithin=title&since=2022-09-16&timeframe=commit&mergegroup=&groupSelect=groupByRepos&checkedFileTypes=docs~functional-code~test-code~other) + * Highlights: Writing the `EditItemDescriptor` class allowed me to learn how to use a defensive copy to prevent + the user from making any unwanted changes to the original copy of the item. +* **New Feature**: Add a multi index class. [#76](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/76) + * What it does: Allows for access of items inside the day list, using a more complex version of an `Index`. + * Justification: There is a need for a different indexing of items that are in the day list compared to the + unscheduled item list. This is to allow users to easily access items in both lists from the same item-planning + page. + * Highlights: It was challenging coming up with a way to represent the different indexing that would satisfy our + needs. +* **New Feature**: Add a plan and unplan command. [#85](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/85), [#87](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/87) + * What it does: Allows for scheduling and unscheduling of items in the wishlist, to a specific day and time. + * Justification: As a travel planning application, users will need to schedule their activities in order to create a + cohesive itinerary using our app. + * Highlights: There was plenty of collaboration and discussion with team members on where to handle each aspect of + the plan and unplan commands, such as shifting the inner workings to the `Itinerary` and `Day` classes. + + + +* **Code contributed**: [RepoSense link](https://nus-cs2103-ay2223s1.github.io/tp-dashboard/?search=seox123&sort=groupTitle&sortWithin=title&timeframe=commit&mergegroup=&groupSelect=groupByRepos&breakdown=true&checkedFileTypes=docs~functional-code~test-code~other&since=2022-09-16&tabOpen=true&tabType=zoom&zFR=false&zA=seox123&zR=AY2223S1-CS2103T-W11-4%2Ftp%5Bmaster%5D&zACS=153.1875&zS=2022-09-16&zFS=seox123&zU=2022-11-07&zMG=false&zFTF=commit&zFGS=groupByRepos) * **Project management**: - * Managed releases `Waddle v1.2` and `v1.2.1` (2 releases) on GitHub + * Managed releases `Waddle v1.4.1` and `1.4.2` (2 releases) on GitHub * **Enhancements to existing features**: - * to be added soon + * Add a `Priority` field to `Item` class. [#61](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/61) + * Fix bugs and close issues. [#100](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/100), [#200](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/200), [#201](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/201), [#202](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/202) + * Write test cases. [#53](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/53), [#228](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/228), [#230](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/230), [#241](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/241) * **Documentation**: * User Guide: - * Added documentation for the features + * Updated user guide with the newly-added commands like `plan`, `unplan`, `free`, and more. [#117](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/117) + * Updated Ui snapshot of the application. [#120](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/120) * Developer Guide: - * Added design details of the Model class - ([#56](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/56), - [#57](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/57)) - * Added Model class UML diagram ([#59](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/59)) + * Added multiple use cases to the developer guide. [#51](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/51) + * Added new proposal to unplan items. [#67](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/67) + +* **Team-based tasks**: + * Conducted testing of app and opened issues. [#101](https://github.com/AY2223S1-CS2103T-W11-4/tp/issues/101), [#102](https://github.com/AY2223S1-CS2103T-W11-4/tp/issues/102) + * Made necessary changes and renaming of classes to be more suitable with our application. [#53](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/53), [#85](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/85), [#248](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/248) * **Community**: * Reported bugs and suggestions for other teams in the class - -* **Tools**: - * Integrated a third party library (Apache PDFBox) to the project - ([#97](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/97), - [#118](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/118), - [#126](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/126)) - -* _{you can add/remove categories in the list above}_ diff --git a/src/main/java/seedu/waddle/MainApp.java b/src/main/java/seedu/waddle/MainApp.java index 7cef2998fa0..82628adef99 100644 --- a/src/main/java/seedu/waddle/MainApp.java +++ b/src/main/java/seedu/waddle/MainApp.java @@ -36,7 +36,7 @@ */ public class MainApp extends Application { - public static final Version VERSION = new Version(1, 3, 2, true); + public static final Version VERSION = new Version(1, 4, 1, true); private static final Logger logger = LogsCenter.getLogger(MainApp.class); From f17129c6672a79a5b41caa54762d06bae82034db Mon Sep 17 00:00:00 2001 From: seox123 Date: Mon, 7 Nov 2022 23:15:29 +0800 Subject: [PATCH 2/4] Remove community section --- docs/team/seox123.md | 3 --- 1 file changed, 3 deletions(-) diff --git a/docs/team/seox123.md b/docs/team/seox123.md index 8b10da2e99d..63f6ba07d12 100644 --- a/docs/team/seox123.md +++ b/docs/team/seox123.md @@ -52,6 +52,3 @@ Given below are my contributions to the project. * **Team-based tasks**: * Conducted testing of app and opened issues. [#101](https://github.com/AY2223S1-CS2103T-W11-4/tp/issues/101), [#102](https://github.com/AY2223S1-CS2103T-W11-4/tp/issues/102) * Made necessary changes and renaming of classes to be more suitable with our application. [#53](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/53), [#85](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/85), [#248](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/248) - -* **Community**: - * Reported bugs and suggestions for other teams in the class From be703dd0faf08e3124e43de4f2c870a44e6744bf Mon Sep 17 00:00:00 2001 From: seox123 Date: Mon, 7 Nov 2022 23:21:19 +0800 Subject: [PATCH 3/4] Make more changes to ppp --- docs/team/seox123.md | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/docs/team/seox123.md b/docs/team/seox123.md index 63f6ba07d12..f935af9483f 100644 --- a/docs/team/seox123.md +++ b/docs/team/seox123.md @@ -39,7 +39,7 @@ Given below are my contributions to the project. * **Enhancements to existing features**: * Add a `Priority` field to `Item` class. [#61](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/61) * Fix bugs and close issues. [#100](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/100), [#200](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/200), [#201](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/201), [#202](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/202) - * Write test cases. [#53](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/53), [#228](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/228), [#230](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/230), [#241](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/241) + * Write test cases, focussing mainly on `Command` and `Parser` classes. [#53](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/53), [#228](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/228), [#230](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/230), [#241](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/241) * **Documentation**: * User Guide: @@ -52,3 +52,6 @@ Given below are my contributions to the project. * **Team-based tasks**: * Conducted testing of app and opened issues. [#101](https://github.com/AY2223S1-CS2103T-W11-4/tp/issues/101), [#102](https://github.com/AY2223S1-CS2103T-W11-4/tp/issues/102) * Made necessary changes and renaming of classes to be more suitable with our application. [#53](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/53), [#85](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/85), [#248](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/248) + +* **Community**: + * Assisted other teams in detection and reporting of bugs during the PE dry run. \ No newline at end of file From af212556af31ad8f1433ed46ce5bea532083af7e Mon Sep 17 00:00:00 2001 From: seox123 Date: Mon, 7 Nov 2022 23:23:53 +0800 Subject: [PATCH 4/4] Fix checkstyle --- docs/team/seox123.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/team/seox123.md b/docs/team/seox123.md index f935af9483f..3819e13d3c2 100644 --- a/docs/team/seox123.md +++ b/docs/team/seox123.md @@ -54,4 +54,4 @@ Given below are my contributions to the project. * Made necessary changes and renaming of classes to be more suitable with our application. [#53](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/53), [#85](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/85), [#248](https://github.com/AY2223S1-CS2103T-W11-4/tp/pull/248) * **Community**: - * Assisted other teams in detection and reporting of bugs during the PE dry run. \ No newline at end of file + * Assisted other teams in detection and reporting of bugs during the PE dry run.