Query parameters not in router state when navigating with back button.

[ngbrown]

I am running the most recent version, v12.1.1 and items that are parameters (?q=test) show up in state.router.query when navigating directly by url, or when pushing state. However, when the back button is used after navigating to another route in the app, the state.router.query doesn't exist on state.router.

This is how I'm connecting to the parameters:

const mapStateToProps = (state, ownProps) => {
  return {
    query: state.router.query.q
  }
};

export default connect(mapStateToProps)(Search)

Guarding for the existence of state.router.query doesn't help because it never shows up when using the back button.

This is what the Redux debugger shows when navigating forward to a link with a parameter:

This is what it shows when navigating back to the same page:

[ngbrown]

When using the back button, the action initiated on store-enhancer.js#L67 doesn't include the query object while the action does contain query for other types of navigation.

At a minimum it looks like the following needs added:

+import qs from 'query-string';
...
    history.listen(newLocation => {
      /* istanbul ignore else */
      if (newLocation) {
        matchCache.clear();
+        newLocation.query = qs.parse(newLocation.search);
        store.dispatch(locationDidChange({
          location: newLocation,
          matchRoute
        }));
      }
    });

[tptee]

This is fixed in my WIP PR here: #130

I'll ping when it's released!

[ngbrown]

Were you able to incorporate any of the other #128 items? For example, I had also improved the tests and the coverage up to 100% by using jsdom.changeURL() instead of faking the getLocation override option.

[tptee]

Fixed a little early in v12.1.2! The solution in #130 is different (we avoid a double parse).