Adding query string doesn't trigger update

[OKNoah]

Description

When I add a query string to the URL, the app does not update.

<Link to="/?hello" />
// or
this.props.history.push('/') // removing a querystring

Environment

Gatsby version: 1.1.28
Node.js version: 8.9.1
Operating System: macOS

Actual result

shouldComponentUpdate (nextProps) {
 // this does not execute at all when query string changes
}

Expected behavior

The component should update

Steps to reproduce

1. Use Link or history to go to /?someString

2. Observe that component does not update.

[calcsam]

Should be fixed when #3516 is merged

[OKNoah]

As a workaround I used this.forceUpdate and the global location. But that meant if I triggered an update in a child component I had to pass forceUpdate down through props.

@calcsam Thanks!

[KyleAMathews]

Due to the high volume of issues, we're closing out older ones without recent activity. Please open a new issue if you need help!