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-T10-4] MatchForGood #54

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

Conversation

simkianseng
Copy link

@simkianseng simkianseng commented Feb 21, 2024

MatchForGood streamlines volunteer management, facilitating efficient scheduling, and coordination of volunteers, ultimately saving time and resources for organizations focused on social impact.

Copy link

codecov bot commented Feb 21, 2024

Codecov Report

Attention: Patch coverage is 67.67956% with 234 lines in your changes are missing coverage. Please review.

Project coverage is 74.55%. Comparing base (b896eb0) to head (e23387f).

❗ Current head e23387f differs from pull request most recent head e7beef8. Consider uploading reports for the commit e7beef8 to get more accurate results

Files Patch % Lines
...ava/seedu/address/ui/AutoCompleteComboBoxPlus.java 0.00% 115 Missing ⚠️
src/main/java/seedu/address/ui/PersonCard.java 0.00% 15 Missing ⚠️
...ain/java/seedu/address/ui/AssignmentListPanel.java 0.00% 12 Missing ⚠️
src/main/java/seedu/address/ui/MainWindow.java 0.00% 12 Missing ⚠️
src/main/java/seedu/address/ui/AssignmentCard.java 0.00% 9 Missing ⚠️
...u/address/storage/JsonSerializableAddressBook.java 52.94% 4 Missing and 4 partials ⚠️
...u/address/logic/commands/AddAssignmentCommand.java 75.86% 2 Missing and 5 partials ⚠️
.../java/seedu/address/model/util/SampleDataUtil.java 0.00% 7 Missing ⚠️
src/main/java/seedu/address/ui/CommandBox.java 0.00% 7 Missing ⚠️
...va/seedu/address/logic/commands/ExportCommand.java 77.77% 1 Missing and 5 partials ⚠️
... and 14 more
Additional details and impacted files
@@             Coverage Diff              @@
##             master      #54      +/-   ##
============================================
- Coverage     75.26%   74.55%   -0.71%     
- Complexity      419      653     +234     
============================================
  Files            71       96      +25     
  Lines          1338     2150     +812     
  Branches        126      245     +119     
============================================
+ Hits           1007     1603     +596     
- Misses          301      484     +183     
- Partials         30       63      +33     

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

@simkianseng simkianseng changed the title [CS2103-T10-4] MatchForGood [CS2103T-T10-4] MatchForGood Feb 21, 2024
tayziyi and others added 28 commits March 31, 2024 12:29
Changed the execute(undo) label in a sequence diagram

Diagram was for the copy command, so executing undo was the wrong label

Lets make sure our labels are correct
Change DG to reflect proposed Copy Command
Using the copy command, we can extract all emails in a comma-seperated format

Previously we were unable to obtain the contact details in a single command

Lets make it easier to copy all contact details and contact our volunteers
…olunteerPrefs

Revert "Change tags to better fit our app"
Allow the filtered list of emails to be copied to the clipboard

Previously, the filtered list of emails was only printed to the command line

Lets copy it to to the clipboard so it can be pasted easily
…the issue of resetting input when textfield loses focus
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.

5 participants