Permalink
Browse files

Merge pull request #11 from tmeasday/master

Actually stop subscriptions (!!)
  • Loading branch information...
svasva committed Nov 1, 2013
2 parents 98213e9 + 30a81f4 commit e2620c9b01dfed3a940f5ca9a09be47410682e50
Showing with 3 additions and 3 deletions.
  1. +3 −3 publish_with_relations.coffee
@@ -54,8 +54,8 @@ Meteor.publishWithRelations = (params) ->
handle.stop() for handle in associations[id]
pub.removed(collection._name, id)
pub.ready() unless params._noReady
pub.onStop ->
for association in associations
handle.stop() for handle in association
for id, association of associations
handle.stop() for key, handle of association
collectionHandle.stop()

4 comments on commit e2620c9

@dweldon

This comment has been minimized.

Show comment
Hide comment
@dweldon

dweldon Nov 1, 2013

Wow. Can we get a version bump or is more testing needed?

dweldon replied Nov 1, 2013

Wow. Can we get a version bump or is more testing needed?

@rschaerer

This comment has been minimized.

Show comment
Hide comment
@rschaerer

rschaerer Dec 28, 2013

@erundook: Vitaly, thank you very much for this meteor package! I'm using it successfully (and with pleasure ;-)) on one of my projects.
However, as @dweldon stated, pushing the changes in the atmosphere package would be nice.

rschaerer replied Dec 28, 2013

@erundook: Vitaly, thank you very much for this meteor package! I'm using it successfully (and with pleasure ;-)) on one of my projects.
However, as @dweldon stated, pushing the changes in the atmosphere package would be nice.

@tmeasday

This comment has been minimized.

Show comment
Hide comment
@tmeasday

tmeasday Apr 8, 2014

Contributor

@svasva -- yikes ! didn't realise you hadn't published this! Can I +1 this -- the package is very broken (don't use it!) without this.

Contributor

tmeasday replied Apr 8, 2014

@svasva -- yikes ! didn't realise you hadn't published this! Can I +1 this -- the package is very broken (don't use it!) without this.

@dweldon

This comment has been minimized.

Show comment
Hide comment
@dweldon

dweldon Apr 8, 2014

@tmeasday Indeed. I've been telling people on stackoverflow to use a local version until/if this gets published. Also see #12

dweldon replied Apr 8, 2014

@tmeasday Indeed. I've been telling people on stackoverflow to use a local version until/if this gets published. Also see #12

Please sign in to comment.