Skip to content

Conversation

@bakobagassas
Copy link
Contributor

Fixes #1618
Remove labor meeting and bonner hours from volunteer hours generated.

Changes

  • Added the isService condition to all the volunteers data.

Testing

  • Go to Create Event
  • When you create a Bonner event and add hours, they don't show up on the reports
Screenshot 2025-09-05 130150
  • When you add another program with the tag "earns service credit" then the hours and event name and other details should show up in the report.
Screenshot 2025-09-05 130136

@github-actions
Copy link

github-actions bot commented Sep 5, 2025

View Code Coverage

Copy link
Contributor

@JohnCox2211 JohnCox2211 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@ojmakinde
Copy link
Contributor

This logic in this PR was implemented by @BrianRamsay as part of a larger change in #1624. Therefore, I'll close this one. Amazing work though! I liked the approach.

@ojmakinde ojmakinde closed this Sep 11, 2025
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.

Volunteer hours generated need to be changed

6 participants