Skip to content

Conversation

@catamphetamine
Copy link
Contributor

remix-run/react-router@a76efdd
@acdlite without this fix this library won't work with react-router@3 which is now the default version

remix-run/react-router@a76efdd
@acdlite without this fix this library won't work with `react-router@3` which is now the default version
@catamphetamine
Copy link
Contributor Author

catamphetamine commented Jan 4, 2017

For those coming here from Google the temporary fix is:

const createRouterObject = require('react-router/lib/RouterUtils').createRouterObject
require('react-router/lib/RouterUtils').createRouterObject = function(history, transitionManager, state = {})
{
  return createRouterObject(history, transitionManager, state)
}

const ReduxRouter = require('redux-router').ReduxRouter

...

@AlexNisnevich
Copy link

@acdlite Any progress in getting this merged? It's annoying to have to use @halt-hammerzeit 's workaround to support react-router v3.

@adamduren
Copy link

@acdlite Any update on this? The last commit was close to a year ago.. Is this project no longer maintained?

@AlexNisnevich
Copy link

I ended up switching to react-router v4 and reorganizing my codebase to remove the need for redux-router.

I think that redux-router is no longer really necessary, and perhaps that's part of the reason it hasn't been maintained in the past year.

@catamphetamine
Copy link
Contributor Author

catamphetamine commented May 5, 2017

@acdlite Any update on this? The last commit was close to a year ago.. Is this project no longer maintained?

@AlexNisnevich It is. You should have dropped it a long time ago.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants