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

Publish an array of cursors #716

Closed
wants to merge 6 commits into from
Closed

Publish an array of cursors #716

wants to merge 6 commits into from

Conversation

@krizka
Copy link

@krizka krizka commented Feb 20, 2013

New version of #682 with all (I hope) remarks fixed.

  • Added some tests for 'several cursors of one collection published' and 'not a cursor in array
  • Added few lines of docuemntation.
    Please feel free to fix documentation or error messages, English is not my native language as you can notice )
Meteor.publish('multi', function (num) {
    return [ Foo.find({ num: num }), Bar.find({ num: num }) ];
});
glasser added a commit that referenced this pull request Mar 5, 2013
- History.md update

- tweak docs

- refactor the "one per collection" check

- make errors into internal errors.  Programming errors like returning the wrong
  type from a function on the server don't need to be reported to the client.
@glasser
Copy link
Member

@glasser glasser commented Mar 5, 2013

Thanks! I reviewed this, made a few tweaks, and merged to devel! This will be in the next Meteor release.

@glasser glasser closed this Mar 5, 2013
@krizka
Copy link
Author

@krizka krizka commented Mar 18, 2013

Haven't seen this before. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.