Tutorial ends with error "You should not use <Link> outside a <Router>"

[dzoudzou]

Tutorial ends with an error.
You should not use <Link> outside a <Router>

[eligao]

I just noticed the bug only appears on desktop view:

On a phone or when the window is small enough, this page renders as a mobile view and doesn't suffer from this bug.

[MiMo42]

Same here with my project.
yarn upgrade
grabs the v4.4.0 release of react-router as a dependency for react-admin.
Dependency hell...

I had to change the react-router dependency NOT to use the version 4.4.0 of react-router.
Using react-router 4.4.0 my app sends lots of warnings about deprecated require statements to the console and also ends with You should not use <Link> outside a <Router> error.

The release tagged as "Latest release" (it's the 4.3.1) of react-router does work.

[fzaninotto]

what's the "Tutorial ends with an error."? Can you please provide a sample project triggering the issue?

Also, since it seems to be a regression on the react-router side, did any of the commenter here open an issue in the react-router repository?

[vaiulian]

Been having this issue all morning.

so this weekend it seems react-router and react-router-dom changed versions:

remix-run/react-router#6630

"You should not use react-router-dom version 4.3.1 with react-router 4.4.0."

i managed to bypass this issue using this remix-run/react-router#6630 (comment)

[vaiulian]

There is also an issue with the history@4.8.0 - deprecating

var createHistory = require('history/createMemoryHistory');

Instead, use:

var createHistory = require("history").createMemoryHistory;

https://github.com/ReactTraining/history/releases

So my big "FIX" is something like

"resolutions": { "react-router": "4.2.0", "react-router-dom": "4.2.0", "history": "4.7.2" }, for react-admin@2.7.0

Still some changes to react-admin core needs to be done.

[nagad814]

Please fix this breaking change; I am getting ready for a demo. Would probably downgrade or try working around.

[eligao]

what's the "Tutorial ends with an error."? Can you please provide a sample project triggering the issue?

Also, since it seems to be a regression on the react-router side, did any of the commenter here open an issue in the react-router repository?

Check out my example (https://codesandbox.io/s/14nl4pw6k4) from my issue #3019

[fzaninotto]

The react-router team seems to have fixed the problem on their side.

remix-run/react-router#6630

Can you please remove your node_modules, run yarn again and check if it fixes the bug?

[MiMo42]

@fzaninotto
Yes, this fixed the bug for me.
Thanks!

[MiMo42]

@vaiulian
I added Issue #3028 for the warnings due to deprecated requires of history@4.8.0,
as that one is not directly related to the problem this issue has been created for originally.