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

Modularizing pages #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Modularizing pages #2

wants to merge 1 commit into from

Conversation

shilpan
Copy link

@shilpan shilpan commented Feb 12, 2014

Makes it clear clear what's a page and what's a view. Pages can probably reuse the views.

@andreypopp
Copy link
Owner

Hi, thanks for the PR. I plan to include several variants of react project templates within this repo:

  • client rendering only
  • client and server rendering (already there)
  • + model layer (via cortex)
  • dynamic code loading (separate bundle for each route)
  • direct data access during server rendering (so we don't have to do XHR requests from server to get data)
  • ...

After the reorganisation I'll merge your PR.

@shilpan
Copy link
Author

shilpan commented Feb 13, 2014

I found issues with using Cortex and react-async btw, I would love to know how you did server side rendering (using react-async) with Cortex as your model

@Shadowfaxenator
Copy link

@andreypopp
Are you going to implement the changes you mentioned? Especially interested in:
direct data access during server rendering
model layer (via cortex)
dynamic code loading

@andreypopp
Copy link
Owner

@Shadowfaxenator yes, but not sure about the ETA

@SimonDegraeve
Copy link

@andreypopp any update on "I plan to include several variants of react project templates within this repo"?
Look forward to check it. I am working with React for couple of months now, and I have to say that your packages were very helpful. I like the techno but still struggle sometimes how to architect my apps, especially with models.
Do you plan to make an example for Flux or Capacitor?

By the way I just created a clone of this repo made with coffee, if you want to check react-coffee-quickstart. If you still plan to create different variant, I could help translating them to coffee.

Thanks

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.

4 participants