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

MobX roadmap (> 2.3) #323

Closed
3 of 7 tasks
mweststrate opened this issue Jun 14, 2016 · 12 comments
Closed
3 of 7 tasks

MobX roadmap (> 2.3) #323

mweststrate opened this issue Jun 14, 2016 · 12 comments

Comments

@mweststrate
Copy link
Member

mweststrate commented Jun 14, 2016

For the previous one see #219 :)

@mweststrate
Copy link
Member Author

Proposal for standardized (de)serialization: Feel free to shoot at it! (Probably it is easier to start with the examples at the bottom) https://gist.github.com/mweststrate/4c9a8e0411ae547fdfe1bc3633dc599e

Might become a stand alone package, not really mobx related so far

@mdebbar
Copy link

mdebbar commented Jun 14, 2016

I tend to agree with @pkieltyka about being careful not to bloat the library with features. The great thing about mobx now is the very short learning curve. It has few concepts that are easy to grasp. Once a lot of concepts are added to mobx, it becomes more difficult to learn. I would suggest trying to keep a minimal API in mobx core, and let libraries provide more features on top of it.

@benjamingr
Copy link
Member

I would like to suggest that we remove features from the library and keep it small, currently there are few concepts I have to learn to use it - optimally there would be even fewer:

  • Doing fromJS is just assigning to an observable property, I don't think it's needed.
  • Derived arrays sounds great, especially since it's a performance boost that's not user facing in any way.
  • I don't think @observable should be extended further, I'd like to see other primitives in libraries and built on top of it.

I'd like to see the library smaller and see how things like Atom Reaction autorun and others be further reduced into a smaller surface API.

@mweststrate
Copy link
Member Author

For the second bullet: See pull request #65 , build 3.4.0-beta.1 can be used to test and give feedback on the new Provider mechanism. Docs: https://github.com/mobxjs/mobx-react/blob/provider/README.md#provider-experimental

(First one is WIP as well)

@mweststrate
Copy link
Member Author

Provider / context support is now released as mobx-react@3.4.0

@jorilallo
Copy link

@mweststrate awesome, I was waiting this to land and works perfectly - such a great addition to separating stores for better testing

@vonovak
Copy link
Contributor

vonovak commented Jul 4, 2016

+1 for (de)serialization 👍

@mweststrate
Copy link
Member Author

Closing this issue as it tends to become a double administration

@jbrantly
Copy link

Apologies for necroing an issue, but I couldn't find any other information after searching. The first item here "provide standardized (de)serialization mechanism" is checked. Does that mean something for that feature has been implemented somewhere?

@mweststrate
Copy link
Member Author

yes, see serializr package (or mobx-state-tree, but that is WIP)

Op do 17 nov. 2016 07:17 schreef James Brantly notifications@github.com:

Apologies for necroing an issue, but I couldn't find any other information
after searching. The first item here "provide standardized
(de)serialization mechanism" is checked. Does that mean something for that
feature has been implemented somewhere?


You are receiving this because you modified the open/close state.

Reply to this email directly, view it on GitHub
#323 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABvGhExAGvQNFJPtLEdTXKpQduM8FNMWks5q-_FsgaJpZM4I1PHJ
.

@gessichen
Copy link

Do we have any samples regarding onBecome(Un)Observed ?

@hccampos
Copy link

I'd just like to point out here for anyone that happens to read this issue that so far serializr has been working great for us. Started our serialization stuff with cerialize (https://github.com/weichx/cerialize) but ended up switching to serializr for the nicer api, better compatibility with MobX and support for references and lookups. Cerialize does provide a nice utility for automatically converting keys to/from e.g. snake_case and camelCase, but that's nothing that can't easily be done in serializr with some custom serialization functions.

Also, computeds and createTransformer can then be used to create a state tree kinda like mobx-state-tree.

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

8 participants