meteorhacks / zones

Zone.js integration for meteor
MIT License
30 stars 11 forks source link

Hijack connection closed #3 #4

Closed thani-sh closed 10 years ago

arunoda commented 10 years ago

Machan, add all these into a single file. Seems like too much :)

thani-sh commented 10 years ago

How about one file for

arunoda commented 10 years ago

There are not much things to hijack. So single file would be great.

thani-sh commented 10 years ago

Moved all hijacks to hijack.js

arunoda commented 10 years ago

Not mini-mongo machan. It's livedata package.


Arunoda Susiripala

@arunoda http://twitter.com/arunoda http://gplus.to/arunodahttps://github.com/arunoda http://www.linkedin.com/in/arunoda

On Thu, Jul 10, 2014 at 1:32 PM, Muhammed Thanish notifications@github.com wrote:

I'm starting on Collection.___ callbacks. Any tips. I was checking out mongo-livedata and minimongo packages

— Reply to this email directly or view it on GitHub https://github.com/meteorhacks/meteor-zone/pull/4#issuecomment-48575632.

arunoda commented 10 years ago

nice.


Arunoda Susiripala

@arunoda http://twitter.com/arunoda http://gplus.to/arunodahttps://github.com/arunoda http://www.linkedin.com/in/arunoda

On Thu, Jul 10, 2014 at 1:41 PM, Muhammed Thanish notifications@github.com wrote:

Moved all hijacks to hijack.js

— Reply to this email directly or view it on GitHub https://github.com/meteorhacks/meteor-zone/pull/4#issuecomment-48576334.

arunoda commented 10 years ago

Does meteor.call working without any issues?

thani-sh commented 10 years ago

Refactored Connection.prototype.apply to use arguments instead of if-else ladder.