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-4] ModuleMateFinder #73

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

Conversation

bakano98
Copy link

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
Copy link

codecov-commenter commented Feb 19, 2022

Codecov Report

Merging #73 (f2967d0) into master (4f7660a) will decrease coverage by 8.97%.
The diff coverage is 56.29%.

@@             Coverage Diff              @@
##             master      #73      +/-   ##
============================================
- Coverage     72.15%   63.18%   -8.98%     
- Complexity      399      629     +230     
============================================
  Files            70      100      +30     
  Lines          1232     2325    +1093     
  Branches        125      292     +167     
============================================
+ Hits            889     1469     +580     
- Misses          311      747     +436     
- Partials         32      109      +77     
Impacted Files Coverage Δ
src/main/java/seedu/address/MainApp.java 0.00% <0.00%> (ø)
...ain/java/seedu/address/commons/core/BookNames.java 0.00% <0.00%> (ø)
...main/java/seedu/address/commons/core/Messages.java 0.00% <ø> (ø)
...a/seedu/address/logic/commands/ArchiveCommand.java 0.00% <0.00%> (ø)
...va/seedu/address/logic/commands/DeleteCommand.java 100.00% <ø> (ø)
...java/seedu/address/logic/commands/FindCommand.java 100.00% <ø> (ø)
...java/seedu/address/logic/commands/ListCommand.java 100.00% <ø> (ø)
...va/seedu/address/logic/commands/SwitchCommand.java 0.00% <0.00%> (ø)
.../seedu/address/logic/parser/AddressBookParser.java 44.44% <0.00%> (-55.56%) ⬇️
...edu/address/logic/parser/ArchiveCommandParser.java 0.00% <0.00%> (ø)
... and 65 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4f7660a...f2967d0. Read the comment docs.

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