Co-authored-by: Adrian Duffell <9312929+adrianduffell@users.noreply.github.com>
- Make `getIdsFromQuery` consider `0` as a valid id.
Previously `Boolean` filter was removing it from the list.
Fixes https://github.com/woocommerce/woocommerce-admin/issues/6980
- Add a basic set of tests for `getIdsFromQuery`, as they were none.
- Replace `lodash.uniq` with ES6 equivalent.
- Update JSDoc for `getIdsFromQuery`, to specify default value, returned type and description.
history.createHashHistory() is being called whenever this package is
imported, which causes the URL to redirect to a hash. This is
unnecessary in cases where elements of wc-admin are used outside of a
completely controlled/routed page.
This lazy-creates the history object only when it's needed and therefore
avoids the URL redirection unless history features are in use.
* Move filter code to the navigation package
- Moves filter-related code to navigation, so it can be shared between components and the app
- Fixes spelling of “flatten”
- Update README docs with new functions
* Use `get` to prevent errors if options is not an array/is empty
* Update documentation and tests
* Move nav-utils and history to new package
* Update config to build and use new navigation package
* Update all imports to use new navigation package
* Remove unnecessary README
* Update nav-utils path
Tomek Wytrębowicz
Ron Rennick
Jeff Stieler
Albert Juhé Lluveras
Kevin Killingsworth
Kelly Dwan