Skip to content

Commit

Permalink
Merge pull request #340 from ldinghan/branch-ppp-docs
Browse files Browse the repository at this point in the history
Add whitespace to ppp
  • Loading branch information
ldinghan authored Nov 14, 2023
2 parents 118da79 + 74aa4d8 commit a23373e
Showing 1 changed file with 9 additions and 1 deletion.
10 changes: 9 additions & 1 deletion docs/team/ldinghan.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,21 +14,25 @@ Given below are my contributions to the project:
* Highlights: There was some design thought given to how information is stored in the model. To prevent cyclic-reference between students and sessions, we decided to only store student models in sessions, and not sessions in students.
* Credits: No reuse of code or third-party libraries.


* **New Feature**: Added the ability to update a session's remark.
* What it does: Allows Avengers to update a session's remark, leaving insightful comments for reflection purposes, as well as tracking of what was already covered in past sessions.
* Justification: This feature greatly complements the attendance features as Avengers may combine the use of this feature, as well as the attendance features, to easily identify what topic each student has been taught and not taught.
* Credits: No reuse of code or third-party libraries.


* **New Feature**: Added the ability to delete created sessions.
* What it does: Allows Avengers to delete previously created sessions, allowing for errors to be corrected when sessions are accidentally created.
* Justification: This is an important feature enhancement for our product because given the nature of our CLI product, mistakes in inputs are likely, and the ability to delete a wrongly-created session will be important to correct these possibly frequently-occurring situations.
* Credits: No reuse of code or third-party libraries.


* **New Feature**: Added the ability to take attendance of students for each session.
* What it does: Enables Avengers to easily record the attendance of their students for each session, allowing for easy tracking of the Avenger's students' attendance.
* Justification: This feature greatly complements the session feature as each session can now store information regarding students' attendance to identify studio tutorial sessions that they have attended.
* Credits: No reuse of code or third-party libraries.


* **New Feature**: Added the ability to view the attendance of particular student(s).
* What it does: Allows Avengers to retrieve an instant overview of their students' attendance to easily identify any studio tutorial sessions that they may have missed.
* Justification: This feature is a significant enhancement to our product as it provides a quick and convenient way for Avengers to monitor individual students' attendance and ensure they are not missing crucial studio tutorial sessions.
Expand All @@ -38,14 +42,15 @@ Given below are my contributions to the project:

* **Code contributed**: [RepoSense](https://nus-cs2103-ay2324s1.github.io/tp-dashboard/?search=ldinghan&breakdown=true).


* **Project management**:
* Reviewed and approved pull requests.
* Organised and created GitHub issues.
* Updated team [weekly meeting notes](https://docs.google.com/document/d/1QKJGL88ciMlyb5MfS2XyThnuUBGdwFIULFgP9MLTFEA/edit).


* **Enhancements to existing features**:
* Converted `address` fields to `teleHandle` fields since address is not as relevant for our product's use case and Telegram Chat is the primary form of communication.
* Converted `address` fields to `teleHandle` fields since address is not as relevant for our product's use case and Telegram Chat is the primary form of communication.


* **Documentation**:
Expand All @@ -60,6 +65,7 @@ Given below are my contributions to the project:
* Added the use cases for sessions.
* Enhanced the user stories.


* Some examples of UML diagrams:
* Class diagram: ![Session Class Diagram](../images/SessionClassUML.png)
* Sequence diagram: ![TakeAttendance Sequence Diagram](../images/TakeAttendanceSequenceDiagram.png)
Expand All @@ -68,13 +74,15 @@ Given below are my contributions to the project:
* **Review/mentoring contributions**:
* [ldinghan's tp comments](https://nus-cs2103-ay2324s1.github.io/dashboards/contents/tp-comments.html#47-lim-han-ldinghan-37-comments)


* **Tools**:
* Java
* IntelliJ
* Git
* FXML
* PlantUML


* **Contributions beyond the project team**:
* Contributed to module forum page to answer queries on code coverage [here](https://github.com/nus-cs2103-AY2324S1/forum/issues/267#issuecomment-1767768888).
* Found 9 bugs during the PE-D, some of which were highly severe bugs, which helped to point out areas for improvement for another team's product.

0 comments on commit a23373e

Please sign in to comment.