Skip to content

[CS2103T-T13-4] ModuleMateFinder - #73

Open
bakano98 wants to merge 555 commits into
nus-cs2103-AY2122S2:masterfrom
AY2122S2-CS2103T-T13-4:master
Open

[CS2103T-T13-4] ModuleMateFinder#73
bakano98 wants to merge 555 commits into
nus-cs2103-AY2122S2:masterfrom
AY2122S2-CS2103T-T13-4:master

Conversation

@bakano98

Copy link
Copy Markdown

This product is for university students that do not have a fixed form-class but rather module-based classes and classmates that they need to contact and keep track of.

@codecov-commenter

codecov-commenter commented Feb 19, 2022

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 50.14085% with 177 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.18%. Comparing base (3c0ba35) to head (bdfcd7e).

Files with missing lines Patch % Lines
...a/seedu/address/logic/commands/ArchiveCommand.java 0.00% 21 Missing ⚠️
src/main/java/seedu/address/ui/MainWindow.java 0.00% 18 Missing ⚠️
...rc/main/java/seedu/address/ui/StatusBarFooter.java 5.26% 18 Missing ⚠️
.../seedu/address/logic/parser/AddressBookParser.java 0.00% 12 Missing and 1 partial ⚠️
...edu/address/logic/parser/ArchiveCommandParser.java 0.00% 13 Missing ⚠️
...rc/main/java/seedu/address/model/ModelManager.java 65.51% 10 Missing ⚠️
...rc/main/java/seedu/address/logic/LogicManager.java 52.63% 5 Missing and 4 partials ⚠️
src/main/java/seedu/address/MainApp.java 0.00% 8 Missing ⚠️
...java/seedu/address/logic/commands/EditCommand.java 60.00% 4 Missing and 4 partials ⚠️
...va/seedu/address/logic/commands/StackUndoRedo.java 78.94% 7 Missing and 1 partial ⚠️
... and 21 more
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##             master      #73      +/-   ##
============================================
- Coverage     66.11%   63.18%   -2.94%     
- Complexity      584      629      +45     
============================================
  Files            92      100       +8     
  Lines          2072     2325     +253     
  Branches        253      292      +39     
============================================
+ Hits           1370     1469      +99     
- Misses          616      747     +131     
- Partials         86      109      +23     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

bakano98 and others added 29 commits March 29, 2022 18:29
Things done are detailed in #113
Fix empty table background colour
Implement Switch and Archive features.

Basic Switch and Archive features work in this PR.
* Able to successfully toggle between two sets of AddressBooks
* Able to Archive a Contact, from the current AddressBook to the Alternate one.

As this feature set is considerably larger than a typical 'normal' command, I will merge this working PR first then update iteratively.
Update SwitchMenuItem and resultDisplay to reflect appropriate AddressBook
Update SwitchMenuItem and resultDisplay
weiming and others added 30 commits April 11, 2022 11:30
Update PPP to comply with 2 page limit
Fix giant formatting of acknowledgement

Cleanup repeated testcase wordings

Improve formatting
Figma mockup drastically different from what we have, so removing it.
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