Meteor-Community-Packages/meteor-roles

[v2.0] support onlyAssigned option for method getUsersInRole

Open

#177 aperta il 1 feb 2016

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)JavaScript (161 fork)batch import
enhancementgood first issuehelp wanted

Metriche repository

Star
 (919 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Use case In my model, 'project.member' is a sub-role of 'project.manager'. If I want a list of all project members that are not managers, AFAIK I cannot yet. Well yes, I can filter the output of getUsersInRole but that's not convenient.

Solution The method getRolesForUser already support this option :

onlyAssigned: return only assigned roles and not automatically inferred (like subroles)

So basically, the idea is to filter users which role param has an assigned:true field, perhaps using Roles._onlyAssignedMatcher method.

Guida contributor

Direzione di ricerca
Controlla l'implementazione di `getRolesForUser` per vedere come viene gestita l'opzione `onlyAssigned`, in particolare l'uso di `Roles. onlyAssignedMatcher`. Applica uno schema simile al metodo `getUsersInRole` nello stesso file sorgente. Aggiorna anche i test rilevanti per coprire la nuova opzione.
Tech stack
javascript
Dominio
authorization
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
familiarity with Meteor roles package
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60