Skip to content

138 unused filters shouldnt be sent to beacon services#147

Open
LarieSB wants to merge 4 commits intodevelopfrom
138-unused-filters-shouldnt-be-sent-to-beacon-services
Open

138 unused filters shouldnt be sent to beacon services#147
LarieSB wants to merge 4 commits intodevelopfrom
138-unused-filters-shouldnt-be-sent-to-beacon-services

Conversation

@LarieSB
Copy link
Copy Markdown
Collaborator

@LarieSB LarieSB commented Oct 14, 2024

What's in the PR

  • Send only the search parameters selected by the user during the search request.
  • The search filters should be deselected by default.

How to test manually

  • Go to Resource Discovery
  • Check the search filters if they are not selected by default.
  • select some filters and execute a search
  • Check in the console that only the selected search filters are sent to the backend.

…p/vp-dp-frontend into 138-unused-filters-shouldnt-be-sent-to-beacon-services
-Remove the gender parameter. There was already a sex parameter.
Send only the search parameters selected by the user during the search request.

The search filters are unselected by default.
@LarieSB LarieSB requested a review from vabishaa October 14, 2024 09:20
@LarieSB LarieSB self-assigned this Oct 14, 2024
@LarieSB LarieSB linked an issue Oct 14, 2024 that may be closed by this pull request
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.

Unused filters shouldn't be sent to Beacon services

1 participant