Consider replacing redux in favor of react context about phenomic HOT 18 CLOSED

I'd really say it depends on what you try to achieve. statinamic already seems opinionated on some parts, and I see it more like a boilerplate than a library. if you want to provide a way to manage data across components whatever the tree structure is in this boilerplate, then I'd say keep redux. If you want it to be more of a library, keep it minimal, and just use your own provider and decorator.

from phenomic.

Statinamic is a collection of lib + a boilerplate generator + docs: you start with a boilerplate that use statinamic libs to help you generate a website with react and webpack. So Statinamic is opininated because of these 2 things (react and webpack). Except that nothing is mandatory (no required structures etc).
You can use the webpack config you want, your own loader, and any npm module you want in your components.

There is a minimal boilerplate that is required to allow some flexibility (choose your styles flavor (inline, css modules, sass...) and I am currently writing a helper to generate the minimal required boilerplate.

The idea is to allow to create a website very easily with one command $ statinamic setup: this will generate the boilerplate that use Statinamic libs. The only things that Statinamic uses are:

• a registry to access page data
• a registry to get available pages layouts (component references) that can be used from md header

Not a lot of things, no user interaction. That's what I was thinking about using react context instead of redux. Currently there is already a context provider to share some metadata about your project (eg: package.json to get homepage url, stuff like that) and I was thinking about using it for all the shared (read only - or sort of) data. The only interaction is: you click on a link, if page data are not in the store, we fetch a json to get those.

from phenomic.

I use redux in almost every project so using redux is one the reasons why I use statinamic.

Also with redux, I can just add reducers that listen for default action of statinamic. Some example use cases: show a loading indicator when getting new page's info. show a error message when getting new page's info failed, ...

In the other hand, keeping redux would make statinamic too opinionated.

One more thing, if you decide to keep redux, you might need to use immutable.js (or similars) because redux store is holding all pages info which could lead to a performance issue on large page.

from phenomic.

show a loading indicator when getting new page's info. show a error message when getting new page's info failed, ...

I think thoses things should be possible without redux (and more easy to implement without using redux).

I will try to avoid redux and maybe will consider removing it.

from phenomic.

I think we should consider this again. I don't see the benefit of using redux anymore. We aren't modify anything in the state (just add more page).

This should be a good alternative solution
https://github.com/heroku/react-refetch

from phenomic.

I think we should not need anything if we remove redux. All the logic for downloading is in PageContainer, maybe we can make a simple fetch/registry directly in here for now. And that might help for hot loading md.

from phenomic.

It is your call. Removing redux would help us reduce the bundle size

from phenomic.

Totally. That's why I opened this issue in the first place :)

from phenomic.

So. Shall we change status of this issue ?

from phenomic.

Done :)

from phenomic.

I think that implement this would be easy, just mutate a window object. 💃

from phenomic.

An alternative http://madx.me/articles/a-simpler-alternative-to-flux-and-redux.html poke @madx

from phenomic.

I was investigating this tool to use with a MobX store app, so removing Redux is 👍 from me.

from phenomic.

Will be handled soon (hopefully)

from phenomic.

Any updates on this issue? It's really nice to have easy way to add your preferred state store. I'm also using mobx currently, and would like to add it to my static site app as well. It's not such simple to replace redux right now.
Thanks.

from phenomic.

I will try to work on that for the next release. Should not take that long.

from phenomic.

👍 thanks for the update

from phenomic.

Going to be fixed with #925

from phenomic.