search

Meteor-Community-Packages / meteor-roles

Authorization package for Meteor, compatible with built-in accounts packages
http://meteor-community-packages.github.io/meteor-roles/
MIT License
921 stars 166 forks source link

Pull request to implement issue 298 #299

Closed jamgold closed 4 years ago

jamgold commented 4 years ago

I modified the _getUsersInRoleCursor method to recognize onlyScoped option as well and added a server side test

SimonSimCity commented 4 years ago

I'll do a more thorough look at this after the weekend, but as of now it looks well from a quick view.

lgtm-com[bot] commented 4 years ago

This pull request fixes 2 alerts when merging 32c230ded9cb92b27a3485d143cabf2732fef186 into 84736398bac15cfc5c1e677f9ede1bfc91602296 - view on LGTM.com

fixed alerts: