🐛 Update Router to 1. always take in location, 2. use location.href i…

[michenly]

…f location is an object

Description

To reproduce the error

1. load up https://github.com/Shopify/gestalt-web-application-example
2. Goto / url to see

3. Goto `/test123` to see

4. Remove wrapping from `NotFound.tsx` and reload `/test123` 5. See the following page with the incorrect image

and see browser error

To check the error is fix
6. quit gestalt-web-application-example server
6. in quilt folder , run command yarn build && yarn tophat react-router ../gestalt-web-application-example
7. dev s in gestalt-web-application-example server folder
8. reload /test123
9. See the expected image below and no react miss-match error in browser

Type of change

• Patch: Bug (non-breaking change which fixes an issue)
• Minor: New feature (non-breaking change which adds functionality)
• Major: Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

• I have added a changelog entry, prefixed by the type of change noted above (Documentation fix and Test update does not need a changelog as we do not publish new version)

[marutypes]

LGTM!