mozilla-releng / balrog / #4955

90%

main: 89%

push

circleci

Pass the state correctly to the `Link` component (#3548)

This fixes a regression from c3f865c72
where updating react-router silently broke keeping some state across
navigation. The issue is that react router's API changed the state
property from being nested within the `to` object to being directly on
the routing components and the `to` component becoming a string. We
didn't see this at all because of the `Link` wrapper that was doing some
massaging on the `to` property when it wasn't a string, so we were just
silently discarding the state parameter.

I removed said massaging to since we're passing strings to `to`
everywhere now as we should.

Fixes #3547

2162 of 2550 branches covered (84.78%)

Branch coverage included in aggregate %.

5692 of 6213 relevant lines covered (91.61%)

0.92 hits per line