Replacing react-router with react-router-dom
Closed this issue · 2 comments
abhinavsau commented
I see for this issue you removed the react-router-dom
dependency. Since react-router-dom
imports and re-exports all the components of react-router
, isn't it better to replace react-router
with react-router-dom
which would give the NavLink
that you needed for the test. Or is there any use case that I might be missing? Thanks @icd2k3
icd2k3 commented
oh interesting... yea maybe we should replace it here 🤔
abhinavsau commented
Opened a PR for you to review. Let me know if I missed anything.