fix(react-router): stabilizing the encoding of path parameters

[SeanCassiere]

closes #1441

This change only impacts the buildLocation method on the Router class, since it is what generates the final href to be committed.

Overall, this change should have zero-impact for current users, whilst still fixing the original problem.

I've tested this with both CSR and SSR and confirmed this change does not break any functionality. Additionally, the useParams hook still returns the decoded value as expected.

Expand

This is what the `useRouterState({ select: s => s.location })` and `useParams` output looks like.

And here's a video of the changed behaviour compared following what was in the original issue.

Screen.Recording.2024-04-19.at.19.15.59.mov

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 58409f5. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution

---

✅ Successfully ran 2 targets

• nx affected --targets=test:format,test:eslint,test:unit,test:build,build --parallel=3
• nx affected --target=test:types --exclude=examples/**

---

Sent with 💌 from NxCloud.