-
Notifications
You must be signed in to change notification settings - Fork 42
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
Associations? #3
Comments
Hi Saimon, at the moment I declare a method on each side of the association so a
I haven't really thought about proper associations - by "proper" I suppose I mean something like ActiveRecord's declarative syntax - but it would certainly be possible to encapsulate the above logic into less code. What kind of thing were you thinking? |
yeah I was thinking of AR-style. I also do what you're doing right now but I was wondering wether you had intentions of making it more declarative. If you haven't done it by the time I get to it, I'll give it a stab. |
Sounds good to me! |
I'm currently working on this feature. It's still in testing phase, but I'm expecting the first iteration to work like this:
Let me know what you think. It's kinda half-working right now, I need to test it more thoroughly. |
sounds good...I never managed to get time to finish off what I started but it was along these lines... |
cool, looks good. let me know when you have something to look at on github. in the meantime i've reopened the issue so i don't forget. |
I have pushed my branch: |
sorry for my belated response, it looks good. i've recently been thinking about some sort of plugin architecture™ to more easily enable this sort of thing so that you could extend js-model's functionality without having to mess around with the internals - something along the lines of sammy's |
Any plans for rudimentary associations between models? How are you doing this now?
The text was updated successfully, but these errors were encountered: