Skip to content

fix: replace accompanying toggle with multi-select volunteer type accordion#552

Open
nadavosa wants to merge 1 commit into
developfrom
fix/accompanying-filter-use-type-accordion
Open

fix: replace accompanying toggle with multi-select volunteer type accordion#552
nadavosa wants to merge 1 commit into
developfrom
fix/accompanying-filter-use-type-accordion

Conversation

@nadavosa
Copy link
Copy Markdown
Collaborator

Summary

  • Removes the binary "Accompanying" switch from the volunteer filters panel
  • The existing Volunteer Type accordion already contains all 4 types (ACCOMPANYING, REGULAR, EVENTS, REGULAR_ACCOMPANYING) and allows multi-select — it fully replaces the toggle
  • Cleans up: CardsFilter type, defaultVolunteerCardsFilter, serializeFilters, deserializeVolunteerFilters, createFilterItems, createSelectedFilterItemsAsFlatArray

Depends on: need4deed-org/be#558 (BE removes accompanying boolean from schema)

Test plan

  • Volunteer filter panel no longer shows the "Accompanying" switch
  • Selecting "Accompanying" in the Volunteer Type accordion filters correctly
  • Selecting multiple types (e.g. Regular + Accompanying) works
  • Deselecting all types shows all volunteers
  • Other filters (district, language, engagement) still work

🤖 Generated with Claude Code

…ordion

Remove the binary "Accompanying" switch filter. The existing volunteer-type
accordion (ACCOMPANYING, REGULAR, EVENTS, REGULAR_ACCOMPANYING) already
covers the same use-case with finer control — no separate boolean needed.
Cleans up CardsFilter type, default filter constant, serializer,
deserializer, and helpers accordingly.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

1 participant