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

Allow permissions to be checked from Meteor publish functions. #14

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

awwx
Copy link
Contributor

@awwx awwx commented Aug 18, 2013

Meteor.user() and Meteor.userId() can't be called from Meteor.publish
functions. This commit adds an optional userId argument to isAdmin
and hasPermission, allowing permissions to be checked from publish
functions.

awwx added 2 commits August 18, 2013 10:53
The package.js on_test needs to include "accounts-password" for the
tests to see "Accounts".
Meteor.user() and Meteor.userId() can't be called from Meteor.publish
functions.  This commit adds an optional `userId` argument to `isAdmin`
and `hasPermission`, allowing permissions to be checked from publish
functions.
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

Successfully merging this pull request may close these issues.

1 participant