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

SmartCollections Update #10

Closed
arunoda opened this issue Aug 11, 2013 · 7 comments
Closed

SmartCollections Update #10

arunoda opened this issue Aug 11, 2013 · 7 comments

Comments

@arunoda
Copy link
Contributor

arunoda commented Aug 11, 2013

Hi,

I've done few changes to SmartCollection server side. Now SC server side is compatible with the Normal Collections.

In the client side Meteor.SmartCollection == Meteor.Collection. So you don't need treat separately for SmartCollections in the client side.

See: http://goo.gl/S5iAe3

Cheers.

@raix
Copy link

raix commented Aug 11, 2013

Nice - this makes things simpler - what about the method calls? are you overwriting the originals or using the sc prefixed ones?

@arunoda
Copy link
Contributor Author

arunoda commented Aug 11, 2013

Now method calls are also compatible with Meteor.Collection. So no code hijack in the Client Side.

@raix
Copy link

raix commented Aug 11, 2013

I see, you overwrite the server methods (or just write it from scratch) - this resembles groundDB in many ways - would be nice to have better options for creating hooks

@raix
Copy link

raix commented Aug 11, 2013

I think this would take some lines of code off groundDB, cool 👍

@arunoda
Copy link
Contributor Author

arunoda commented Aug 11, 2013

Yes. And one of your comment is previous emails leads to this change :) And I have less code to manage too ;)

@raix
Copy link

raix commented Aug 11, 2013

It's great work @arunoda, it fits nicely together with the autoforms and schemas @aldeed is working on. 👍

@arunoda
Copy link
Contributor Author

arunoda commented Aug 12, 2013

Wow glad to here that.

@raix raix closed this as completed in 9f5c703 Aug 19, 2013
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