Skip to content

Commit

Permalink
Feature/clear filters redux refactored (#79)
Browse files Browse the repository at this point in the history
Summary:
Clear search filters on search when checkbox is checked

Details:
* Add checkbox to AdvancedSearchFilterInput

* Added state for clearFilters

* Added stub for clearing filters in customListEditor handleAddCustomListEditorAdvSearchQuery method

* Clear filters on search when checkbox is checked

* Updated AdvancedSearchFilterInput test

* Add clearFilters property to CustomListEditorAdvancedSearchBuilderData in Redux store

* Cleaned up old clearFilters approach

* Setting clear filters flag

* Added isClearFiltersEnabled to initial state in customListEditor reducer

* Clear filters on search working with global state

* Add node version configuration for nodenv

* Use unique name and label for include/exclude

* Use EditableInput instead of HTML input tag

* Fixed typo and removed unnecessary id property

* Refactored to use searchParams clearFilters keyed by builder name

* Added msw to package.json; Added CustomLists test to confirm clear filters on search works as expected

* Update CustomLists test to reference clear filters checkbox by name, not merely by role; Eliminate extra label tag and use label prop of EditableInput instead
  • Loading branch information
jdempcy authored Apr 4, 2023
1 parent d4d6a50 commit 50422f2
Show file tree
Hide file tree
Showing 16 changed files with 1,042 additions and 15 deletions.
1 change: 1 addition & 0 deletions .node-version
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
18.14.2
Loading

0 comments on commit 50422f2

Please sign in to comment.