-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
React-router 6.0.0-beta.0 throws an error #40
Comments
Hi @Ethorsen, the problem is that v beta has a breaking change from v alpha. |
Ok. No stress thx for the follow-up. I'll stick with react-router 5.2.0 for now as my actual project wont grow that big. It will be trivial to do the switch once its all figured out. Thx |
Does it mean your PR with react-router is to be merged soon? |
import { HistoryRouter as Router } from "redux-first-history/rr6";
import { Route, Routes, Navigate } from "react-router-dom";
//...
import { store, history } from "./store";
//...
<Router history={history}>
<Routes>
<Route path="/dashboard" element={<Dashboard />} />
<Route path="/" element={<Home />} />
<Route path="*" element={<Navigate to="/" />} />
</Routes>
</Router> demo rr6 here: https://codesandbox.io/s/redux-first-history-demo-rr6-uccuw |
Hi @salvoravida,
Thx for this great module.
There is an error right out of the box with the latest react-router beta.
As shown in this sandbox.
https://codesandbox.io/s/proud-hill-e6kn2
Thx
The text was updated successfully, but these errors were encountered: