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

You could use ampersand-state instead of model #2

Closed
HenrikJoreteg opened this issue Jan 11, 2015 · 1 comment
Closed

You could use ampersand-state instead of model #2

HenrikJoreteg opened this issue Jan 11, 2015 · 1 comment
Milestone

Comments

@HenrikJoreteg
Copy link

Hey, this is a really cool and clever use of ampersand stuff! I love that about modular code, people find awesome, unexpected ways to use it.

One suggestion would be that you could just use ampersand-state where you're currently using ampersand-model. It's the same code but model extends state with a few methods and dependencies that are for persisting models via Ajax to a server which don't look like they're being used here.

Anyway, cool stuff! Might be cool to see a few usage examples in the docs too.

Cheers!

@zeropaper
Copy link
Owner

Thanks a lot for the feedback!

Actually.. I was wondering if I should have use states instead of models.

On the other hand, I could imagine a special "sync" to actually CRUD on file system (in node.js) and CRUD over a RESTful API when in the browser.
Probably going to look at that further when I have a bit of time.

About the examples and docs.
I first made that set of classes.. to generate documentation. So obviously, it's going to happen.

Cheers and thanks again!

@zeropaper zeropaper added this to the v0.1.0 milestone Feb 2, 2015
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

2 participants