nus-cs2103-AY2223S1 · ConradLew · Nov 4, 2022 · Nov 4, 2022 · Nov 4, 2022 · Nov 4, 2022
diff --git a/docs/AboutUs.md b/docs/AboutUs.md
@@ -17,7 +17,7 @@ We are a team based in the [School of Computing, National University of Singapor
 * Role: Developer
 * Responsibilities: UI
 
-### Conrad Lew Kian Loong
+### Lew Kian Loong, Conrad
 
 <img src="images/conradlew.png" width="200px">
 

diff --git a/docs/UserGuide.md b/docs/UserGuide.md
@@ -356,6 +356,9 @@ Format: `client -v`
 
 ## Issue Commands
 
+> NOTE: Multiple issues can have the exact same fields (other than the issue id), as it is possible for a project to 
+> have multiple of the same issues.
+
 ### Adding an issue : `issue -a`
 
 Adds an issue to the AddressBook. A unique issue ID will be automatically generated. 

diff --git a/docs/team/aishwarya-hariharan-iyer.md b/docs/team/aishwarya-hariharan-iyer.md
@@ -11,9 +11,7 @@ GitHub repo every time and manually check different pages to see which tasks req
 
 Given below are my contributions to the project.
 
-* **Code Contributed**: The link to the tP dashboard can be found [here](https://nus-cs2103-ay2223s1.github.
-  io/tp-dashboard/?search=aishwarya-hariharan-iyer&breakdown=true&sort=groupTitle&sortWithin=title&since=2022-09-16
-  &timeframe=commit&mergegroup=&groupSelect=groupByRepos&checkedFileTypes=docs~functional-code~test-code~other)
+* **Code Contributed**: The link to the tP dashboard can be found [here](https://nus-cs2103-ay2223s1.github.io/tp-dashboard/?search=aishwarya-hariharan-iyer&breakdown=true&sort=groupTitle&sortWithin=title&since=2022-09-16&timeframe=commit&mergegroup=&groupSelect=groupByRepos&checkedFileTypes=docs~functional-code~test-code~other)
 
 * **New features implemented**: A summary of the enhancements you implemented.
     * **New Feature**: `Client` class with `add`, `delete` and `edit` commands

diff --git a/docs/team/conradlew.md b/docs/team/conradlew.md
@@ -5,6 +5,42 @@ title: Conrad's Project Portfolio Page
 
 ### Project: DevEnable
 
-DevEnable is a product for developers who have to manage different projects spread across multiple GitHub repositories. It helps developers organize information about different projects they are working on in one place so that they may prioritize and have an overview. It removes the hassle of having to navigate to our/organization’s GitHub repo every time and manually check different pages to see which tasks require immediate attention.
+DevEnable is a product for developers who have to manage different projects spread
+across multiple GitHub repositories. It helps developers organize information about
+different projects they are working on in one place so that they may prioritize and
+have a convenient overview of what needs to be done.
 
-To be updated.
+Given below are my contributions to the project.
+
+* **New Feature**: Added the ability to pin/unpin entities (projects, clients and issues)
+    * What it does: Allows the user to pin entities to the top of their respective lists one at a time. Preceding pin commands can be reversed by using the pin command again on the same entity.
+    * Justification: This feature improves the product significantly because a user can value some entities more than others and the application should provide a way for users to keep the more important entities easily visible.
+    * Highlights: This enhancement affected the existing sort commands. It required an in-depth analysis of design alternatives. The implementation was challenging as it required changes to the existing sort commands.
+
+* **New Feature**: Added a list command that allows the user to navigate to the different entity lists commands using the CLI.
+
+* **New Feature**: Added a default view command that allows the user to choose what entity list they would prefer to see on opening the application.
+
+* **Code contributed**: [RepoSense link](https://nus-cs2103-ay2223s1.github.io/tp-dashboard/?search=conradlew&breakdown=true&sort=groupTitle&sortWithin=title&since=2022-09-16&timeframe=commit&mergegroup=&groupSelect=groupByRepos&checkedFileTypes=docs~functional-code~test-code~other)
+
+* **Project management**:
+    * Managed releases `v1.3.trial` - `v1.3` (2 releases) on GitHub
+
+* **Enhancements to existing features**:
+    * Updated the storage, model and UI to support display of multiple entities (Pull requests [\#64](https://github.com/AY2223S1-CS2103-F13-1/tp/pull/64), [\#65](https://github.com/AY2223S1-CS2103-F13-1/tp/pull/65))
+    * Wrote additional tests for implemented features to increase coverage (Pull requests [\#104](https://github.com/AY2223S1-CS2103-F13-1/tp/pull/104), [\#133](https://github.com/AY2223S1-CS2103-F13-1/tp/pull/133))
+
+* **Documentation**:
+    * User Guide:
+        * Added documentation for the `pin` feature [\#141](https://github.com/AY2223S1-CS2103-F13-1/tp/pull/141).
+    * Developer Guide:
+        * Added implementation details of the `list`, `default view` and `pin` features [\#106](https://github.com/AY2223S1-CS2103-F13-1/tp/pull/106).
+
+* **Contributions to team-based tasks**:
+  * Set up team repository
+  * Managed issue tracker
+  * Created demo video for v1.2
+
+* **Community**:
+    * PRs reviewed (with non-trivial review comments): [\#30](https://github.com/AY2223S1-CS2103-F13-1/tp/pull/30), [\#80](https://github.com/AY2223S1-CS2103-F13-1/tp/pull/80), [\#62](https://github.com/AY2223S1-CS2103-F13-1/tp/pull/62)
+    * Reported bugs and suggestions for other teams in the class [(Link)](https://github.com/ConradLew/ped/issues)
diff --git a/docs/team/crvstalphua.md b/docs/team/crvstalphua.md
@@ -5,6 +5,51 @@ title: Crystal's Project Portfolio Page
 
 ### Project: DevEnable
 
-DevEnable is a product for developers who have to manage different projects spread across multiple GitHub repositories. It helps developers organize information about different projects they are working on in one place so that they may prioritize and have an overview. It removes the hassle of having to navigate to our/organization’s GitHub repo every time and manually check different pages to see which tasks require immediate attention.
+DevEnable is a product for developers who have to manage different projects spread across multiple GitHub repositories.
+It helps developers organize information about different projects they are working on in one place, making it easier 
+for them to prioritize and have an overview of their projects, issues and clients. It helps them avoid the hassle of 
+having to manually navigate to different Github repositories to check which tasks require immediate attention.
 
-To be updated.
+Given below are my contributions to the project.
+
+* **Code Contributed**: The link to the tP dashboard can be found [here](https://nus-cs2103-ay2223s1.github.io/tp-dashboard/?search=crvstalphua&breakdown=true&sort=groupTitle&sortWithin=title&since=2022-09-16&timeframe=commit&mergegroup=&groupSelect=groupByRepos&checkedFileTypes=docs~functional-code~test-code~other&tabOpen=false).
+
+* **New features implemented**: A summary of the enhancements you implemented.
+  * **New Feature**: `Issue` class with `add`, `delete` and `edit` commands
+    * What it does: The `Issue` class represents an issue belonging to a project. Each issue consists of a title, 
+      deadline, status, urgency, pin and a project which it belongs to. Title, deadline, status and urgency fields each
+      have their own classes. The user can create an issue, adding it to the `IssueList`and the list of issues of the 
+      relevant project. The title, deadline, and urgency of an existing issue can also be edited. The user can remove
+      an issue, deleting it from the `IssueList` and the list of issues of the project. A list of all issues can be 
+      viewed through `issue -l` command as well.
+    * Justification: It enables developers to easily add or delete issues to or from each of their projects. Each 
+      issue has its own deadline, status and urgency so that the developer can decide how to prioritize and choose 
+      which issues to work on. The developer can also modify fields if there are changes to the issue without having to 
+      delete and add another issue. 
+    * Highlights: Similar to the `Project` and `Client` classes, issues can be sorted based on various keys or filtered
+      based on specific search terms, as well as pinned.
+
+  * **New Feature**: `mark` and `unmark` commands in `Issue` class
+    * What it does: When an issue is create, the default status is `incomplete`. The `mark` command changes the status
+      of an issue to `completed` while the `unmark` command changes the status of an issue to `incomplete`.
+    * Justification: Upon completing an issue, the developer can mark it as completed, making it easier to see 
+      which issues still need to be addressed. If the developer realises that a completed issue requires more work, 
+      they can unmark it, changing its status to incomplete.
+    * Highlights: Projects are displayed with the count for number of complete and incomplete issues so that the 
+      developer can see which project has more issues that require attention.
+
+* **Enhancements to existing features**: 
+  * E.g. changes to GUI, wrote additional tests, etc.
+
+* **Documentation**:
+  * Contributions to the UG
+    * Added documentation and images for project command features: `add`, `delete`, `edit`, `list`, `sort`and `set default view`
+    * Added documentation and images for issue command features: `add`, `delete`, `edit`, `list`, `sort`, `mark`, `unmark`, and 
+      `set default view`
+    * Added documentation and images for client command features: `list` and `sort` 
+  * Contrbution to the DG
+    * Write-up for Edit Command Feature consisting of Edit Project Command, Edit Issue Command and Edit Client Command
+    * Design considerations for Edit Command Feature
+
+* **Contributions to the team-based tasks**: 
+  * Reviewed some team pull requests
diff --git a/docs/team/donovan9617.md b/docs/team/donovan9617.md
@@ -11,9 +11,7 @@ GitHub repo every time and manually check different pages to see which tasks req
 
 Given below are my contributions to the project.
 
-* **Code Contributed**: The link to the tP dashboard can be found [here](https://nus-cs2103-ay2223s1.github.
-  io/tp-dashboard/?search=Donovan9617&breakdown=true&sort=groupTitle&sortWithin=title&since=2022-09-16
-  &timeframe=commit&mergegroup=&groupSelect=groupByRepos&checkedFileTypes=docs~functional-code~test-code~other)
+* **Code Contributed**: The link to the tP dashboard can be found [here](https://nus-cs2103-ay2223s1.github.io/tp-dashboard/?search=Donovan9617&breakdown=true&sort=groupTitle&sortWithin=title&since=2022-09-16&timeframe=commit&mergegroup=&groupSelect=groupByRepos&checkedFileTypes=docs~functional-code~test-code~other)
 
 * **New features implemented**: A summary of the enhancements you implemented.
     * **New Feature**: `Project` class with `add`, `delete` and `edit` commands

diff --git a/src/main/java/seedu/address/logic/commands/client/AddClientCommand.java b/src/main/java/seedu/address/logic/commands/client/AddClientCommand.java
@@ -28,8 +28,8 @@ public class AddClientCommand extends ClientCommand {
             + ": Adds a client to the project book. \n"
             + "Parameters: "
             + PREFIX_NAME + "NAME "
-            + PREFIX_MOBILE + "MOBILE "
-            + PREFIX_EMAIL + "EMAIL "
+            + "[" + PREFIX_MOBILE + "MOBILE] "
+            + "[" + PREFIX_EMAIL + "EMAIL] "
             + PREFIX_PROJECT_ID + "PROJECT ID \n"
             + "Example: " + COMMAND_WORD + " "
             + COMMAND_FLAG + " "

diff --git a/src/main/java/seedu/address/logic/commands/client/EditClientCommand.java b/src/main/java/seedu/address/logic/commands/client/EditClientCommand.java
@@ -31,14 +31,16 @@ public class EditClientCommand extends ClientCommand {
 
     public static final String MESSAGE_CLIENT_NOT_FOUND = "Client id %1$d does not exist in the project book";
 
+    public static final String MESSAGE_CLIENT_ALREADY_HAS_THAT_NAME = "This client already has that name";
+
     public static final String MESSAGE_USAGE = COMMAND_WORD
             + " " + COMMAND_FLAG
             + ": Edits a client in the project book. \n"
             + "Parameters: "
-            + PREFIX_NAME + "NAME "
-            + PREFIX_EMAIL + "EMAIL "
-            + PREFIX_MOBILE + "MOBILE "
-            + PREFIX_CLIENT_ID + "ID \n"
+            + "[" + PREFIX_NAME + "NAME] "
+            + "[" + PREFIX_EMAIL + "EMAIL] "
+            + "[" + PREFIX_MOBILE + "MOBILE] "
+            + PREFIX_CLIENT_ID + "CLIENT ID \n"
             + "Example: " + COMMAND_WORD + " "
             + COMMAND_FLAG + " "
             + PREFIX_NAME + "John "
@@ -75,6 +77,9 @@ public CommandResult execute(Model model, Ui ui) throws CommandException {
         if (newName != null) {
             for (Client c : model.getFilteredClientList()) {
                 if (c.getClientName().equals(newName)) {
+                    if (toEditClient.getClientName().equals(newName)) {
+                        throw new CommandException(MESSAGE_CLIENT_ALREADY_HAS_THAT_NAME);
+                    }
                     throw new CommandException(MESSAGE_DUPLICATE_CLIENT_NAME);
                 }
             }

diff --git a/src/main/java/seedu/address/logic/commands/client/FindClientCommand.java b/src/main/java/seedu/address/logic/commands/client/FindClientCommand.java
@@ -24,10 +24,10 @@ public class FindClientCommand extends ClientCommand {
     public static final String MESSAGE_FIND_CLIENT_USAGE = COMMAND_WORD + " " + COMMAND_FLAG
             + ": Finds clients by keyword.\n"
             + "Parameters: "
-            + PREFIX_NAME + "NAME "
-            + PREFIX_CLIENT_ID + "CLIENT ID "
-            + PREFIX_EMAIL + "EMAIL "
-            + PREFIX_MOBILE + "MOBILE \n"
+            + "[" + PREFIX_NAME + "NAME] "
+            + "[" + PREFIX_CLIENT_ID + "CLIENT ID] "
+            + "[" + PREFIX_EMAIL + "EMAIL] "
+            + "[" + PREFIX_MOBILE + "MOBILE] \n"
             + "Example: " + COMMAND_WORD + " "
             + COMMAND_FLAG + " "
             + PREFIX_NAME + "John "

diff --git a/src/main/java/seedu/address/logic/commands/client/SortClientCommand.java b/src/main/java/seedu/address/logic/commands/client/SortClientCommand.java
@@ -24,11 +24,11 @@ public class SortClientCommand extends ClientCommand {
 
     public static final String MESSAGE_USAGE = COMMAND_WORD + ": Sort clients in project book. \n"
             + "Sort by client id: "
-            + PREFIX_CLIENT_ID + "0 (ascending) or "
-            + PREFIX_CLIENT_ID + "1 (descending)."
+            + "[" + PREFIX_CLIENT_ID + "0] (ascending) or "
+            + "[" + PREFIX_CLIENT_ID + "1] (descending)."
             + "Sort by name: "
-            + PREFIX_NAME + "0 (alphabetical) or "
-            + PREFIX_NAME + "1 (reverse alphabetical). \n"
+            + "[" + PREFIX_NAME + "0] (alphabetical) or "
+            + "[" + PREFIX_NAME + "1] (reverse alphabetical). \n"
             + "Example: "
             + COMMAND_WORD + " "
             + COMMAND_FLAG + " "

diff --git a/src/main/java/seedu/address/logic/commands/issue/AddIssueCommand.java b/src/main/java/seedu/address/logic/commands/issue/AddIssueCommand.java
@@ -29,8 +29,8 @@ public class AddIssueCommand extends IssueCommand {
             + "Parameters: "
             + PREFIX_PROJECT_ID + "PROJECT_ID "
             + PREFIX_TITLE + "TITLE "
-            + PREFIX_DEADLINE + "DEADLINE "
-            + PREFIX_URGENCY + "URGENCY(0, 1, 2, 3) \n"
+            + "[" + PREFIX_DEADLINE + "DEADLINE] "
+            + "[" + PREFIX_URGENCY + "URGENCY(0, 1, 2, 3)] \n"
             + "Example: " + COMMAND_WORD + " "
             + COMMAND_FLAG + " "
             + PREFIX_PROJECT_ID + "1 "
@@ -70,6 +70,10 @@ public CommandResult execute(Model model, Ui ui) throws CommandException {
 
         Issue toAdd = toAddWithoutModel.apply(model);
 
+        if (!toAdd.hasValidId()) {
+            throw new CommandException(ID_OVERFLOW);
+        }
+
         if (model.hasIssue(toAdd)) {
             throw new CommandException(MESSAGE_DUPLICATE_ISSUE);
         }

diff --git a/src/main/java/seedu/address/logic/commands/issue/EditIssueCommand.java b/src/main/java/seedu/address/logic/commands/issue/EditIssueCommand.java
@@ -29,9 +29,9 @@ public class EditIssueCommand extends IssueCommand {
             + ": Edits an issue in the project book. \n"
             + "Parameters: "
             + PREFIX_ISSUE_ID + "ISSUE_ID "
-            + PREFIX_TITLE + "TITLE "
-            + PREFIX_DEADLINE + "DEADLINE "
-            + PREFIX_URGENCY + "URGENCY(0, 1, 2, 3) "
+            + "[" + PREFIX_TITLE + "TITLE] "
+            + "[" + PREFIX_DEADLINE + "DEADLINE] "
+            + "[" + PREFIX_URGENCY + "URGENCY(0, 1, 2, 3)] "
             + "Example: " + COMMAND_WORD + " "
             + COMMAND_FLAG + " "
             + PREFIX_ISSUE_ID + "1 "

diff --git a/src/main/java/seedu/address/logic/commands/issue/FindIssueCommand.java b/src/main/java/seedu/address/logic/commands/issue/FindIssueCommand.java
@@ -26,12 +26,12 @@ public class FindIssueCommand extends IssueCommand {
     public static final String MESSAGE_FIND_ISSUE_USAGE = COMMAND_WORD + " " + COMMAND_FLAG
             + ": Finds issues by keyword.\n"
             + "Parameters: "
-            + PREFIX_TITLE + "TITLE "
-            + PREFIX_STATUS + "STATUS "
-            + PREFIX_URGENCY + "URGENCY "
-            + PREFIX_PROJECT_NAME + "PROJECT NAME "
-            + PREFIX_PROJECT_ID + "PROJECT ID "
-            + PREFIX_ISSUE_ID + "ISSUE ID "
+            + "[" + PREFIX_TITLE + "TITLE] "
+            + "[" + PREFIX_STATUS + "STATUS] "
+            + "[" + PREFIX_URGENCY + "URGENCY] "
+            + "[" + PREFIX_PROJECT_NAME + "PROJECT NAME] "
+            + "[" + PREFIX_PROJECT_ID + "PROJECT ID] "
+            + "[" + PREFIX_ISSUE_ID + "ISSUE ID] "
             + "Example: " + COMMAND_WORD + " "
             + COMMAND_FLAG + " "
             + PREFIX_TITLE + "This is an issue "

diff --git a/src/main/java/seedu/address/logic/commands/issue/SortIssueCommand.java b/src/main/java/seedu/address/logic/commands/issue/SortIssueCommand.java
@@ -26,14 +26,14 @@ public class SortIssueCommand extends IssueCommand {
     public static final String MESSAGE_USAGE = COMMAND_WORD + " " + COMMAND_FLAG
             + ": Sort issues in project book. \n"
             + "Sort by issue id: "
-            + PREFIX_ISSUE_ID + "0 (ascending) or "
-            + PREFIX_ISSUE_ID + "1 (descending). "
+            + "[" + PREFIX_ISSUE_ID + "0] (ascending) or "
+            + "[" + PREFIX_ISSUE_ID + "1] (descending). "
             + "Sort by deadline: "
-            + PREFIX_DEADLINE + "0 (chronological) or "
-            + PREFIX_DEADLINE + "1 (reverse chronological). "
+            + "[" + PREFIX_DEADLINE + "0] (chronological) or "
+            + "[" + PREFIX_DEADLINE + "1] (reverse chronological). "
             + "Sort by urgency: "
-            + PREFIX_URGENCY + "0 (ascending) or "
-            + PREFIX_URGENCY + "1 (descending). "
+            + "[" + PREFIX_URGENCY + "0] (ascending) or "
+            + "[" + PREFIX_URGENCY + "1] (descending). "
             + "Example: "
             + COMMAND_WORD + " "
             + COMMAND_FLAG + " "

diff --git a/src/main/java/seedu/address/logic/commands/project/AddProjectCommand.java b/src/main/java/seedu/address/logic/commands/project/AddProjectCommand.java
@@ -27,9 +27,9 @@ public class AddProjectCommand extends ProjectCommand {
             + ": Adds a project to the project book. \n"
             + "Parameters: "
             + PREFIX_NAME + "NAME "
-            + PREFIX_CLIENT_ID + "CLIENT_ID "
-            + PREFIX_REPOSITORY + "REPOSITORY "
-            + PREFIX_DEADLINE + "DEADLINE \n"
+            + "[" + PREFIX_CLIENT_ID + "CLIENT_ID] "
+            + "[" + PREFIX_REPOSITORY + "REPOSITORY] "
+            + "[" + PREFIX_DEADLINE + "DEADLINE] \n"
             + "Example: " + COMMAND_WORD + " "
             + COMMAND_FLAG + " "
             + PREFIX_NAME + "John "