Skip to content

Conversation

wladi0097
Copy link
Contributor

No description provided.

@wladi0097 wladi0097 changed the title FINDO-8454_combine_available_and_selected_filters FINDO-8454_combine_available_and_selected_filters (Step 2: Add the story points) Feb 19, 2020
@wladi0097 wladi0097 changed the base branch from 0.x to FINDO-8454_2nd_combine_available_and_selected_filters February 19, 2020 10:16
…into FINDO-8454_combine_available_and_selected_filters
"multiple"
],
"description": "Whether one or more filter values can be selected at the same time. Only applies to a subset of available filter types."
"description": "Whether one or more filter values can be selected at the same time. Only applies to a subset of available filter types. Filters of select-type single have just one value to select, unless 'cat'-filter and range-filter. Filters of select-type multiple with 'combinationOperation' of type 'or' can have more than one value, each values are OR-combined. Filters of select-type multiple with 'combinationOperation' of type 'and' can have more than one value, each values are AND-combined."
Copy link
Collaborator

Choose a reason for hiding this comment

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

Filters of select-type single have just one value to select, unless 'cat'-filter and range-filter.

cat and range filters are literally the only ones with select type single, right? This sentence sounds contradictory.

Filters of select-type multiple with 'combinationOperation' of type 'and' can have more than one value, each values are AND-combined."

...each value is AND-combined.

Copy link
Contributor Author

@wladi0097 wladi0097 Feb 19, 2020

Choose a reason for hiding this comment

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

This whole thing just has way to many redundant informations. The best explanation would be Whether one or more filter values can be selected at the same time. I think

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes.

Wladislaw Koschemako added 2 commits February 19, 2020 17:07
@howard howard merged commit 1619b0b into FINDO-8454_2nd_combine_available_and_selected_filters Feb 20, 2020
@howard howard deleted the FINDO-8454_combine_available_and_selected_filters branch February 20, 2020 12:27
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.

3 participants