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-F11-2] MODPRO #92

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

Conversation

dlimyy
Copy link

@dlimyy dlimyy commented Sep 16, 2022

MODPRO is a desktop application which helps NUS students in tracking the progress of their modules. It is highly optimised for students who prefer Command Line Interface (CLI) by allowing those who type fast to key in commands to track their modules. It also provides a Graphic User interface(GUI) for those who prefer it.

@codecov-commenter
Copy link

codecov-commenter commented Sep 16, 2022

Codecov Report

Base: 72.15% // Head: 83.85% // Increases project coverage by +11.69% 🎉

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

Additional details and impacted files
@@              Coverage Diff              @@
##             master      #92       +/-   ##
=============================================
+ Coverage     72.15%   83.85%   +11.69%     
- Complexity      399     1121      +722     
=============================================
  Files            70      140       +70     
  Lines          1232     2861     +1629     
  Branches        125      389      +264     
=============================================
+ Hits            889     2399     +1510     
- Misses          311      384       +73     
- Partials         32       78       +46     
Impacted Files Coverage Δ
src/main/java/seedu/address/MainApp.java 0.00% <0.00%> (ø)
...main/java/seedu/address/commons/core/Messages.java 0.00% <ø> (ø)
...del/module/exceptions/NullModuleCodeException.java 0.00% <0.00%> (ø)
src/main/java/seedu/address/ui/ExamCard.java 0.00% <0.00%> (ø)
src/main/java/seedu/address/ui/ExamListPanel.java 0.00% <0.00%> (ø)
src/main/java/seedu/address/ui/HelpWindow.java 0.00% <ø> (ø)
src/main/java/seedu/address/ui/MainWindow.java 0.00% <0.00%> (ø)
src/main/java/seedu/address/ui/ModuleCard.java 0.00% <0.00%> (ø)
...rc/main/java/seedu/address/ui/ModuleListPanel.java 0.00% <0.00%> (ø)
src/main/java/seedu/address/ui/TaskCard.java 0.00% <0.00%> (ø)
... and 107 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.

CFSY pushed a commit to CFSY/tp that referenced this pull request Oct 25, 2022
dlimyy and others added 28 commits November 4, 2022 12:15
Add tests for UnlinkExamCommand and UnlinkExamCommandParser classes
…into add-tests

* 'master' of https://github.com/AY2223S1-CS2103T-F11-2/tp:
  Update PPP
  edit-PPP
  Update ppp
  Update PPP
  PPP added.
  Update grammatical error
  Add PPP
  Added relevant information for PPP.
  Amend tests in TaskTest.java
  rename edit method in task for clarity
  Add tests for UnlinkExamCommand and UnlinkExamCommandParser classes
  Amend incorrect javadocs in Module.java
  Add tests for ParserUtil
  Amend bug for edit module feature
  Fix grammar mistake.
  Add JsonAdaptedExamTest
…into add-tests-for-model

* 'master' of https://github.com/AY2223S1-CS2103T-F11-2/tp:
  Update PPP
  edit-PPP
  Update ppp
  Update PPP
  PPP added.
  Update grammatical error
  Add PPP
  Added relevant information for PPP.
  Amend tests in TaskTest.java
  rename edit method in task for clarity
  Add tests for UnlinkExamCommand and UnlinkExamCommandParser classes
  Amend incorrect javadocs in Module.java
  Amend bug for edit module feature
  Fix grammar mistake.
  Add JsonAdaptedExamTest
* add-tests:
  Correct the failed test for DistinctTaskList
  Add tests for Exam and DistinctTaskList
Updated JsonSerializableAddressBook tests.
Fixed issues with EOF for all json files.
Updated LogicManager tests and removed redundandt AB3 tests.
Removed redundant imports in Logic and LogicManager classes.
Modified json files for JsonAddressBookStorageTest.
* 'master' of https://github.com/AY2223S1-CS2103T-F11-2/tp: (51 commits)
  Update PPP
  edit-PPP
  Update ppp
  Update PPP
  PPP added.
  Update grammatical error
  Add PPP
  Added relevant information for PPP.
  Amend tests in TaskTest.java
  rename edit method in task for clarity
  Add tests for UnlinkExamCommand and UnlinkExamCommandParser classes
  Amend incorrect javadocs in Module.java
  Make some changes based on the comments
  Add tests for ParserUtil
  Amend bug for edit module feature
  Fix grammar mistake.
  Add JsonAdaptedExamTest
  Add invalid index check for parser tests
  Fix checkstyle error and add tests for mark and unmark methods
  Change variable names.
  ...
* master: (97 commits)
  Update PPP
  edit-PPP
  Update ppp
  Update PPP
  PPP added.
  Update grammatical error
  Add PPP
  Added relevant information for PPP.
  Amend tests in TaskTest.java
  rename edit method in task for clarity
  Add tests for UnlinkExamCommand and UnlinkExamCommandParser classes
  Amend incorrect javadocs in Module.java
  Make some changes based on the comments
  Add tests for ParserUtil
  Amend bug for edit module feature
  Fix grammar mistake.
  Add JsonAdaptedExamTest
  Add invalid index check for parser tests
  Fix checkstyle error and add tests for mark and unmark methods
  Change variable names.
  ...
# Conflicts:
#	src/test/java/seedu/address/testutil/TypicalTasks.java
Add tests for DeleteExamCommand and DeleteExamCommandParser classes
# Conflicts:
#	src/test/java/seedu/address/logic/commands/CommandTestUtil.java
Updated architecture diagrams for model, logic, storage.
dlimyy and others added 30 commits November 7, 2022 22:38
Add page break after future features
Update Acknowledgements and table of contents
Made minor tweaks to spacing
Made minor changes to help command description
Changed from note to warning for delete exam
Made more changes to UG
Update jar file link in UG
…into update-ug

* 'master' of https://github.com/AY2223S1-CS2103T-F11-2/tp: (76 commits)
  Update jar file link in UG
  Update DeveloperGuide.md
  Update DeveloperGuide.md
  Update UserGuide.md
  Update UserGuide.md
  Update UserGuide.md
  Update DeveloperGuide.md
  Update UserGuide.md
  Update DeveloperGuide.md
  Update UserGuide.md
  Update UserGuide.md
  Update DeveloperGuide.md
  Update UserGuide.md
  Update UserGuide.md
  Update UserGuide.md
  Update final PPP
  Remove binary file
  Change to addressBookOptional
  Update final PPP
  Update PPP
  ...
Improve consistency and fix formatting error for UG
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