Skip to content
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

Closed
3 tasks
ericelliott opened this issue Dec 9, 2015 · 3 comments
Closed
3 tasks

Universal Routing #1941

ericelliott opened this issue Dec 9, 2015 · 3 comments

Comments

@ericelliott
Copy link
Contributor

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

@LorbusChris
Copy link
Contributor

For implementation, I'm looking at Redux Simple Router. Happy to discuss other options.

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!

@ericelliott
Copy link
Contributor Author

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?

Redux simple router does use React Router, and takes a minimalist approach to it.

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!

Since redux-simple-router is really just a thin integration with React Router, we may be able to support both.

@gautamsi
Copy link
Member

Keystone 4 is going in maintenance mode. Expect no major change. see #4913 for details.
Keystone v5 is using GraphQL extensively and one of the few examples are based on SSR with Next.js for React.

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

No branches or pull requests

4 participants