Merge pull request #271 from ji-just-ji/master

Fix AddParseSequenceDiagram and minor PPP fix.

docs/diagrams/AddParseSequenceDiagram.puml

@@ -16,14 +16,14 @@ AddCommandParser -> ParserUtil : parseModuleCode("CS2040S")
 activate ParserUtil
 ParserUtil --> AddCommandParser : moduleCode:ModuleCode
 
-AddCommandParser -> ParserUtil : parseModuleCode("1")
-ParserUtil --> AddCommandParser : moduleCode:ModuleCode
+AddCommandParser -> ParserUtil : parseYear("1")
+ParserUtil --> AddCommandParser : y:Year
 
-AddCommandParser -> ParserUtil : parseModuleCode("1")
-ParserUtil --> AddCommandParser : moduleCode:ModuleCode
+AddCommandParser -> ParserUtil : parseSemester("1")
+ParserUtil --> AddCommandParser : s:Semester
 
-AddCommandParser -> ParserUtil : parseModuleCode("A")
-ParserUtil --> AddCommandParser : moduleCode:ModuleCode
+AddCommandParser -> ParserUtil : parseGrade("A")
+ParserUtil --> AddCommandParser : g:Grade
 
 deactivate ParserUtil
 

docs/team/ji-just-ji.md

@@ -20,6 +20,7 @@ Given below are my contributions to the project.
   * Enhanced `UniquePersonList` to `UniqueModuleList` to accommodate modules instead of person.
   * Enhanced `Person` object to `Module` object to accommodate for the purpose of the software.
   * Upgraded Json files to accomodate for `Module` objects 
+  * Created tests and `TypicalModules` and `ModuleBuilder` as backbone for testing commands.
 
 * **Documentation**:
   * User Guide:
@@ -30,9 +31,10 @@ Given below are my contributions to the project.
     * `calculateCap` Command: elaboration and cleanup
     * `calculateMCs` Command: elaboration and cleanup
   * Developer Guide:
-    * `Storage` section of Implementation and associated diagrams.
+    * `Module Storage` section of Implementation and overall associated diagrams.
     * `add` command section and associated diagrams.
     * Added use cases UC06 and UC07
+    * Contributed to creation of user stories in the ideation and the development stage.
 
 * **Community**:
   * PRs reviewed (with non-trivial review comments): 4