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

SSR & Immutable Stores? #10

Open
paulwehner opened this issue Oct 30, 2015 · 4 comments
Open

SSR & Immutable Stores? #10

paulwehner opened this issue Oct 30, 2015 · 4 comments

Comments

@paulwehner
Copy link

Was wondering where you are on the ToDo list? Are you including SSR soon and Immutable Stores?

@rozzzly
Copy link

rozzzly commented Oct 30, 2015

I've been using altjs for my flux implementation. It has an iso package for serializing store state and sending that with the ssr render so client immediately has same state. However, it leaves a few things to be desired when integrated with meteor, so I'm merging it and the popular ReactLayout amid a few other cool things I won't go into atm. ImmutableJS is something I've been wanting to do for a while and it should pair nicely with my little project. With my current setup I believe the adoption will be very quick :) This repo helped me alot with getting started, ill be sure to make a fork/branch/pr with my stuffs in a few days.

@AdamBrodzinski
Copy link
Owner

@paulwehner I've stopped working on SSR since arunoda is working on it. I don't have time to add an immutable store for flux but I think that Redux stores can be checked with === because each time the reducer changes data a brand new object is created with a clone and new changes are added to that. I haven't had enough issues to check that though.

Using immutable.js with Alt or Redux will be the same as the guides for non-meteor apps.

Hope this helps!

@rozzzly that sounds great! I'll be more than happy to accept a PR. We'll prob have to put that on a new branch so it doesn't look overly complex for new flux users.

@paulwehner
Copy link
Author

@AdamBrodzinski thanks for the update.

@rozzzly Sounds cool. Will look forward to when you publish it.

@dbackeus
Copy link

@rozzzly looking forward to seeing the PR 👍

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

No branches or pull requests

4 participants