Closed ManpreetSL closed 7 months ago
Awesome work! Thank you for the time 🔥 👍 💯
As a side note: If you're ready to open a PR already, I don't think it makes much sense to open an issue just prior to the PR. The issues are more for tracking purposes and communication I think.
Awesome work! Thank you for the time 🔥 👍 💯
As a side note: If you're ready to open a PR already, I don't think it makes much sense to open an issue just prior to the PR. The issues are more for tracking purposes and communication I think.
:) Ahh yeah okay. In the future, I'll just open PRs when it's work I've done like this, then.
Summary
React Router DOM package is changing things around in v6. See here for more details. 2 issues that we've identified are:
useHistory()
hook from the React Router package, but the TS branch has v6 installed, so we'd like to update usages of the old function to its replacement,useNavigate()
wherever possible.<Redirect />
component is used, but v6 uses<Navigate />
insteadApproach
useHistory()
useNavigate()
functionuseNavigate()
'snavigate
function instead ofuseHistory()
's.push()
<Redirect />
component<Redirect />
with<Navigate />