- Notifications
You must be signed in to change notification settings - Fork429
[How to add NgRx] - NgRx Store & App State handling#259
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:master
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
pjmagee commentedJun 28, 2017
Where would you rehydrate the store? |
estebanvega commentedJul 8, 2017
@pjmagee Could you usehttps://github.com/btroncone/ngrx-store-localstorage to rehydrate from localStorage. Should work with angular 4.x as well. |
estebanvega commentedJul 8, 2017
@hakonamatata@MarkPieszak would it be interesting to see an example of an XHR request to an REST API, which in turn updates the client store via something likehttps://github.com/ngrx/effects? |
pjmagee commentedJul 10, 2017 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@estebanvega@MarkPieszak but what about SSR since ngrx-store-localstorage has browser dependencies? Also what about authentication? When a user is authenticated and the site is refreshed, then authentication is lost from the store...and....do youreally want to store that information in local storage? :) Actually,, SSR seems so pointless right now, many third party components will just break if you have SSR enabled. Things like PrimeNG etc. |
A lot of those decisions (on what to pass down from server->client) and such you'd need to decide for your project what's important to do that with, and what you can live without rendering on the server (and let it just happen on the client). PrimeNG unfortunately entangled their library into the DOM instead of using Renderer and other abstractions provided :/ On the bright side though! We are working on adding a mocking layer over the window/document/etc so that it will be more flexible and hopefully better accommodate libraries that are accidentally (or not) using the DOM, so that they will still be able to Render in a server-render. 👍 |
Added ngrx/store
Take and remove what you like@MarkPieszak 😉