Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CS2103T-T13-1] TeachingAssistantAssistant #96

Open
wants to merge 222 commits into
base: master
Choose a base branch
from

Conversation

tensaida
Copy link

Teaching Assistant Assistant (TAA) is a desktop app for Teaching Assistants (TAs). It keeps track of TAs’ students, tutorial groups, and tasks.

@codecov-commenter
Copy link

codecov-commenter commented Sep 28, 2022

Codecov Report

Base: 72.15% // Head: 55.33% // Decreases project coverage by -16.82% ⚠️

Coverage data is based on head (e10a110) compared to base (77a32bf).
Patch coverage: 47.25% of modified lines in pull request are covered.

Additional details and impacted files
@@              Coverage Diff              @@
##             master      #96       +/-   ##
=============================================
- Coverage     72.15%   55.33%   -16.83%     
- Complexity      399      566      +167     
=============================================
  Files            70      115       +45     
  Lines          1232     2409     +1177     
  Branches        125      299      +174     
=============================================
+ Hits            889     1333      +444     
- Misses          311      991      +680     
- Partials         32       85       +53     
Impacted Files Coverage Δ
src/main/java/seedu/address/MainApp.java 0.00% <ø> (ø)
...main/java/seedu/address/commons/core/Messages.java 0.00% <ø> (ø)
...ava/seedu/address/logic/commands/ClearCommand.java 100.00% <ø> (ø)
...java/seedu/address/logic/commands/ExitCommand.java 100.00% <ø> (ø)
...java/seedu/address/logic/commands/HelpCommand.java 100.00% <ø> (ø)
...address/logic/commands/grade/GradeViewCommand.java 0.00% <0.00%> (ø)
...ss/logic/commands/student/StudentExpelCommand.java 0.00% <0.00%> (ø)
...ic/commands/student/StudentResetFilterCommand.java 0.00% <0.00%> (ø)
...u/address/logic/commands/task/TaskEditCommand.java 0.00% <0.00%> (ø)
...u/address/logic/commands/task/TaskListCommand.java 0.00% <0.00%> (ø)
... and 83 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

tensaida and others added 30 commits November 5, 2022 14:12
* Add Grade Design Consideration to DG

* Placate checkstyle
* Added Task Tests

* Minor fix
* Add Grade Design Consideration to DG

* Placate checkstyle

* Upload UML Diagram images
* Update UML Diagrams to match current iteration of TAA

* Fix hanging div tag
* Implement Student List Command

* Add skeleton file for Student Edit Command

* Fix styling

* Fix styling

* Implement StudentEditCommand

* Fix styling

* Fix styling

* Fix styling

* Remove address from StudentEditCommand

* Fix styling

* Fix the issue of StudentAddCommand not accepting a TutorialGroup parameter

* Implement StudentListCommandTest

* Fix styling

* Fix styling

* Update student edit message

* Fix styling

* Fix styling

* Update PPP
* Update UML Diagrams to match current iteration of TAA

* Fix hanging div tag

* Fix Grade notes in UG
* Fix student not updated in task bug

* Fix grade reset bug

* Placate checkstyle

* Fix test files

Co-authored-by: limweijun <[email protected]>
…uide.md`. (#242)

* Clarify valid email address format

* Change `t/<tag>` to `t/<tags>`
* Implement Student List Command

* Add skeleton file for Student Edit Command

* Fix styling

* Fix styling

* Implement StudentEditCommand

* Fix styling

* Fix styling

* Fix styling

* Remove address from StudentEditCommand

* Fix styling

* Fix the issue of StudentAddCommand not accepting a TutorialGroup parameter

* Implement StudentListCommandTest

* Fix styling

* Fix styling

* Update student edit message

* Fix styling

* Fix styling

* Add sequence diagram for tutorial filter

* Remove tutorial group constructor
* Update dg

* Update dg

* Fix checkstyle

* Fix issues found in review

* Change size of TaskClassDiagram in DG

* Add info about task sort

Co-authored-by: Devesh <[email protected]>
Co-authored-by: tensaida <[email protected]>
* Update dg

* Update dg

* Fix checkstyle

* Fix issues found in review

* Change size of image

* Add composition relationship to TaskClassDiagram in DG

Co-authored-by: limweijun <[email protected]>
Co-authored-by: limweijun <[email protected]>
* Update DG

* Update DG

* Add diagrams

* Update appendix
* Fix documentations in classes

* Remove the non-empty field line in javadocs
* Fix ordering

* Minor fixes

* Minor fixes

* Minor fixes

* Remove AB3 reference when printing

* Removed placeholder text

* Format
* Update limweijun.md

* Update ppp
* Implement Student List Command

* Add skeleton file for Student Edit Command

* Fix styling

* Fix styling

* Implement StudentEditCommand

* Fix styling

* Fix styling

* Fix styling

* Remove address from StudentEditCommand

* Fix styling

* Fix the issue of StudentAddCommand not accepting a TutorialGroup parameter

* Implement StudentListCommandTest

* Fix styling

* Fix styling

* Update student edit message

* Fix styling

* Fix styling

* Update PPP

* Update tutorial group feature in DG

* Solve conflict
* Implement Student List Command

* Add skeleton file for Student Edit Command

* Fix styling

* Fix styling

* Implement StudentEditCommand

* Fix styling

* Fix styling

* Fix styling

* Remove address from StudentEditCommand

* Fix styling

* Fix the issue of StudentAddCommand not accepting a TutorialGroup parameter

* Implement StudentListCommandTest

* Fix styling

* Fix styling

* Update student edit message

* Fix styling

* Fix styling

* Update PPP

* Update PPP
* Add test cases to GradeTest.java

* Add test cases to GradeTest.java
* Update PPP

* Update PPP
* Update PPP

* Update PPP

* Update Grade DG

* Add image file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants