-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Universal Routing #1941
Comments
I've been wondering how much this would opinionate Keystone as a framework. How about simply using React Router, leaving the choice of further react router integration up to the App? Personally I'd be interested in a Relay/GraphQL Implementation using React-Router-Relay for my website's public (/frontend/non-admin) UI. Relay will hopefully soon become universal/isomorphic: facebook/relay#589 , this could be great! |
Redux simple router does use React Router, and takes a minimalist approach to it.
Since redux-simple-router is really just a thin integration with React Router, we may be able to support both. |
Keystone 4 is going in maintenance mode. Expect no major change. see #4913 for details. |
I need Universal Routing for my apps, and it looks like it is already under discussion. I figured this might be a good place to keep track of implementation details & collect feedback.
It seems to me that routing should be a pretty simple drop-in replacement in most apps. I thought at first there might be a lot of hooks into the routing system for middleware, etc, but a cursory glance at the code only reveals a few potential breaking spots. If you know of others, please chime in:
cursory glance over potential breaks:
Are there more middleware hooks I should be aware of?
There's also a test getKeystone helper that will probably need fixing.
For implementation, I'm looking at Redux Simple Router. Happy to discuss other options.
See also:
cc @JedWatson
The text was updated successfully, but these errors were encountered: