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

[CS2103-W12-2] VolunSync #77

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

Conversation

JovenSoh
Copy link

@JovenSoh JovenSoh commented Sep 19, 2024

VolunSync helps nonprofits and community organisations manage volunteers, track hours, and coordinate events. It is optimised for volunteer-focused HR management, allowing for efficient scheduling, tracking, and communication through both CLI and GUI.

@JovenSoh JovenSoh marked this pull request as ready for review September 19, 2024 09:13
Copy link

codecov bot commented Sep 19, 2024

Codecov Report

Attention: Patch coverage is 74.41077% with 76 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...lunteer/VolunteerIsAvailableForEventPredicate.java 28.57% 12 Missing and 3 partials ⚠️
src/main/java/seedu/address/model/AddressBook.java 59.37% 9 Missing and 4 partials ⚠️
.../seedu/address/model/volunteer/VolunteerDates.java 74.41% 6 Missing and 5 partials ⚠️
...rc/main/java/seedu/address/model/EventManager.java 9.09% 9 Missing and 1 partial ⚠️
...r/eventcommandparser/EventFilterCommandParser.java 0.00% 6 Missing ⚠️
...rc/main/java/seedu/address/model/ModelManager.java 80.00% 1 Missing and 4 partials ⚠️
...ain/java/seedu/address/model/VolunteerManager.java 16.66% 4 Missing and 1 partial ⚠️
src/main/java/seedu/address/model/event/Event.java 0.00% 1 Missing and 2 partials ⚠️
...model/event/EventNameContainsKeywordPredicate.java 83.33% 1 Missing and 1 partial ⚠️
...VolunteerIsAssignedToUnfreeDayTargetException.java 0.00% 2 Missing ⚠️
... and 3 more
Files with missing lines Coverage Δ Complexity Δ
...va/seedu/address/logic/commands/AssignCommand.java 96.66% <100.00%> (ø) 9.00 <0.00> (?)
...gic/commands/eventcommands/EventFilterCommand.java 100.00% <100.00%> (ø) 7.00 <7.00> (?)
.../logic/commands/eventcommands/EventNewCommand.java 100.00% <100.00%> (ø) 8.00 <1.00> (?)
...logic/commands/eventcommands/EventViewCommand.java 100.00% <ø> (ø) 7.00 <0.00> (?)
...logic/commands/eventcommands/FindEventCommand.java 100.00% <100.00%> (ø) 8.00 <2.00> (?)
...mmands/volunteercommands/FindVolunteerCommand.java 100.00% <100.00%> (ø) 8.00 <2.00> (?)
...nds/volunteercommands/VolunteerAddDateCommand.java 100.00% <100.00%> (ø) 7.00 <0.00> (?)
...ands/volunteercommands/VolunteerDeleteCommand.java 100.00% <ø> (ø) 7.00 <0.00> (?)
...ommands/volunteercommands/VolunteerNewCommand.java 100.00% <100.00%> (ø) 7.00 <0.00> (?)
.../volunteercommands/VolunteerRemoveDateCommand.java 100.00% <100.00%> (ø) 7.00 <0.00> (?)
... and 29 more

... and 8 files with indirect coverage changes

juliantayyc pushed a commit to juliantayyc/tp that referenced this pull request Oct 27, 2024
keerthigkaarthik and others added 27 commits November 7, 2024 02:23
…thik/tp into fixAddandRemoveDate

* 'fixAddandRemoveDate' of https://github.com/keerthigkaarthik/tp:
  Update README.md
  Fix VolunteerDates
  Add various volunteer parser tests and fixed bugs in parsers
  Style fix
  Add VolunterNewCommandParser test cases and did some bug fixes
  Fix codestyle issue and bug
  Use static command indicators
  Fix codestyle issue
  Prevent loading overlapping events from json
  Remove list of matches from find command result
  Disallow empty description if 'des/' is present
  Modify find command error prompt
  Reorder event details on GUI
  Improve readability of add command prompt
  Modify time error prompt
  Fix text format error
  Change example prompt of delete commands
  Update new event cli message
  Fix Assign and Unassign error message
* Created testcases for JsonAdaptedEvent
* Added testcases for EventTest
This is a working commit, passes all test cases.
tzuennn and others added 30 commits November 12, 2024 12:31
Update string validation appendix
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