Sadd visual (#244)

* Update javadocs, consistency on sessions to use small s

* Add picture for sadd

Author: benclmnt

docs/UserGuide.md

@@ -502,8 +502,16 @@ This helps to prevent you from accidentally agreeing to 2 sessions that overlaps
 
 </div>
 
+<figure style="width:auto; text-align:center; padding:0.5em; font-style: italic; font-size: smaller;">
+    <p>
+        <img src="images/sadd_sample.png" style="width: 100%; height: auto;"/>
+    </p>
+    <figcaption>Figure - Result of executing <code>sadd g/New Gym ex/Endurance at/06/11/2020 0900 t/65</code></figcaption>
+</figure>
+
 Examples:
-* `sadd g/Machoman Gym ex/Endurance at/29/09/2020 1600 t/120` adds a session at gym `Machoman` with exercise type `Endurance` at `29/09/2020 1600hrs` that lasts for `120` minutes
+* `sadd g/New Gym ex/Endurance at/06/11/2020 0900 t/65` adds a session at gym `New Gym` with exercise type `Endurance` at `06/11/2020 0900hrs` that lasts for `65` minutes. 
+If the session is within the [viewing period](#334-viewing-sessions-within-period-sview) of the Session List, it will be displayed on the right panel of the UI.
 
 
 ### 3.3.2 Editing a Session : `sedit`

docs/images/sadd_sample.png

@@ -0,0 +1,49 @@
+---
+layout: page
+title: Bennett Clement's Project Portfolio Page
+---
+
+## Project: FitEgo
+
+FitEgo is a desktop application for fitness instructors to schedule, and keep track of his/her customers' progress in one place.
+
+Given below are my contributions to the project.
+
+* **New Feature**: Added the ability to add / delete sessions.
+  * What it does: allows the user to create and delete fitness sessions
+  * Justification: This feature is core to the product because the user needs to create a session before they can schedule a client.
+  * Highlights: This enhancement required an in-depth analysis of design alternatives.
+  The implementation was challenging as it deals with dates, the notion of "overlapping" dates, and handling schedules related to the session
+
+* **New Feature**: Added table to view a list of client's schedules
+  * What it does: List client's schedules (with remarks, payment status and exercise type)
+  * Justification: This feature makes it easy for the user to keep track of client's progress in one page
+  * Highlights: The implementation was challenging as it required the table to be resizable and to show updates as soon as user changes any details about the client / schedules.
+
+* **Code contributed**: [RepoSense link](https://nus-cs2103-ay2021s1.github.io/tp-dashboard/#breakdown=true&search=&sort=groupTitle&sortWithin=title&since=2020-08-14&timeframe=commit&mergegroup=&groupSelect=groupByRepos&checkedFileTypes=docs~functional-code~test-code~other&tabOpen=true&tabType=authorship&tabAuthor=benclmnt&tabRepo=AY2021S1-CS2103T-T13-3%2Ftp%5Bmaster%5D&authorshipIsMergeGroup=false&authorshipFileTypes=test-code)
+
+* **Project management**:
+  * Managed all releases `v1.1` - `v1.3` (4 releases) on GitHub
+  * Setting up the GitHub team org/repo
+  * Setting up tools (Gradle)
+  * Maintained issue tracker by setting up milestones, label and triaging bugs
+
+* **Enhancements to existing features**:
+  * Wrote tests for Session model (Pull requests [\#140](https://github.com/AY2021S1-CS2103T-T13-3/tp/pull/140))
+
+* **Documentation**:
+  * User Guide:
+    * Added documentation for the features `sadd` and `sdel` [\#139](https://github.com/AY2021S1-CS2103T-T13-3/tp/pull/139)
+    * Added explanation on common terminologies used in FitEgo User Guide [\#139](https://github.com/AY2021S1-CS2103T-T13-3/tp/pull/139)
+  * Developer Guide:
+    * Added implementation details of the `sdel` feature.
+
+* **Review/mentoring contributions**:
+  * Regularly discuss implementation details in our group's communication channel.
+  * Help others debug and find solution during team meetings.
+  * Post issues and discussions on issue tracker. (Some examples: 
+  [\#98](https://github.com/AY2021S1-CS2103T-T13-3/tp/issues/98), [\#107](https://github.com/AY2021S1-CS2103T-T13-3/tp/issues/107), [\#223](https://github.com/AY2021S1-CS2103T-T13-3/tp/issues/223))
+
+* **Tools**:
+  * Integrated a new Github plugin (PlantUML) to the team repo
+

docs/team/benclmnt.md

@@ -7,9 +7,9 @@ public class Messages {
 
     public static final String MESSAGE_UNKNOWN_COMMAND = "Unknown command";
     public static final String MESSAGE_INVALID_COMMAND_FORMAT = "Invalid command format! \n%1$s";
-    public static final String MESSAGE_INVALID_CLIENT_DISPLAYED_INDEX = "The Client index provided is invalid";
-    public static final String MESSAGE_INVALID_SESSION_DISPLAYED_INDEX = "The Session index provided is invalid";
-    public static final String MESSAGE_INVALID_SCHEDULE_DISPLAYED_INDEX = "The Schedule index provided is invalid";
-    public static final String MESSAGE_CLIENTS_LISTED_OVERVIEW = "%1$d Clients listed!";
+    public static final String MESSAGE_INVALID_CLIENT_DISPLAYED_INDEX = "The client index provided is invalid";
+    public static final String MESSAGE_INVALID_SESSION_DISPLAYED_INDEX = "The session index provided is invalid";
+    public static final String MESSAGE_INVALID_SCHEDULE_DISPLAYED_INDEX = "The schedule index provided is invalid";
+    public static final String MESSAGE_CLIENTS_LISTED_OVERVIEW = "%1$d clients listed!";
 
 }

src/main/java/seedu/address/commons/core/Messages.java

@@ -18,7 +18,7 @@
 public class AddSessionCommand extends Command {
     public static final String COMMAND_WORD = "sadd";
 
-    public static final String MESSAGE_USAGE = COMMAND_WORD + ": Adds a Session to FitEgo. "
+    public static final String MESSAGE_USAGE = COMMAND_WORD + ": Adds a session to FitEgo. "
             + "Parameters: "
             + PREFIX_GYM + "GYM "
             + PREFIX_EXERCISE_TYPE + "EXERCISE_TYPE "
@@ -30,9 +30,9 @@ public class AddSessionCommand extends Command {
             + PREFIX_START_TIME + "29/09/2020 1600 "
             + PREFIX_DURATION + "120 ";
 
-    public static final String MESSAGE_SUCCESS = "New Session added: %1$s";
-    public static final String MESSAGE_DUPLICATE_SESSION = "This Session overlaps with an existing Session in FitEgo.\n"
-            + "A Session is considered overlapping if another Session starts before the current Session ends.";
+    public static final String MESSAGE_SUCCESS = "New session added: %1$s";
+    public static final String MESSAGE_DUPLICATE_SESSION = "This session overlaps with an existing session in FitEgo.\n"
+            + "A session is considered overlapping if it starts before the previous session ends.";
 
     private final Session toAdd;
 

src/main/java/seedu/address/logic/commands/session/AddSessionCommand.java

@@ -20,14 +20,14 @@ public class DeleteSessionCommand extends Command {
     public static final String COMMAND_WORD = "sdel";
 
     public static final String MESSAGE_USAGE = COMMAND_WORD
-            + ": Deletes the Session identified by the index number used in the displayed Session list.\n"
+            + ": Deletes the session identified by the index number used in the displayed Session List.\n"
             + "Parameters: INDEX (must be a positive integer)\n"
             + "Example: " + COMMAND_WORD + " 1";
 
-    public static final String MESSAGE_DELETE_SESSION_SUCCESS = "Deleted Session: %1$s";
+    public static final String MESSAGE_DELETE_SESSION_SUCCESS = "Deleted session: %1$s";
 
     public static final String MESSAGE_FORCE_DELETE_SESSION_USAGE = COMMAND_WORD
-            + ": Cannot delete the Session identified by the index number because there are schedules associated to it."
+            + ": Cannot delete the session identified by the index number because there are schedules associated to it."
             + "\nTo force delete, pass in f/ as an option. BEWARE, YOU WILL LOSE ALL RELATED SCHEDULES."
             + "\nParameters: INDEX (must be a positive integer) f/"
             + "\nExample: " + COMMAND_WORD + " 1 f/";

src/main/java/seedu/address/logic/commands/session/DeleteSessionCommand.java

@@ -25,8 +25,8 @@ public class EditSessionCommand extends Command {
 
     public static final String COMMAND_WORD = "sedit";
 
-    public static final String MESSAGE_USAGE = COMMAND_WORD + ": Edits the details of the Session identified "
-            + "by the index number used in the displayed Session list.\n"
+    public static final String MESSAGE_USAGE = COMMAND_WORD + ": Edits the details of the session identified "
+            + "by the index number used in the displayed Session List.\n"
             + "Existing values will be overwritten by the input values.\n"
             + "Parameters: INDEX (must be a positive integer) "
             + "[" + PREFIX_GYM + "GYM] "
@@ -38,10 +38,10 @@ public class EditSessionCommand extends Command {
             + PREFIX_START_TIME + "29/09/2020 1600 "
             + PREFIX_DURATION + "120 ";
 
-    public static final String MESSAGE_EDIT_SESSION_SUCCESS = "Edited Session: %1$s";
+    public static final String MESSAGE_EDIT_SESSION_SUCCESS = "Edited session: %1$s";
     public static final String MESSAGE_NOT_EDITED = "At least one field to edit must be provided.";
-    public static final String MESSAGE_DUPLICATE_SESSION = "This Session already exists in FitEgo.";
-    public static final String MESSAGE_OVERLAPPING_SESSION = "This Session overlaps with another "
+    public static final String MESSAGE_DUPLICATE_SESSION = "This session already exists in FitEgo.";
+    public static final String MESSAGE_OVERLAPPING_SESSION = "This session overlaps with another "
             + "session in the FitEgo.";
 
     private final Index index;