Meteor-Community-Packages/meteor-roles

Internal function _getInheritedRoleNames() should return role object

Ouverte

#304 ouverte le 10 déc. 2019

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)JavaScript (161 forks)batch import
enhancementgood first issue

Métriques du dépôt

Stars
 (919 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

https://github.com/Meteor-Community-Packages/meteor-roles/blob/32c230ded9cb92b27a3485d143cabf2732fef186/roles/roles_common.js#L543-L557

The function _getInheritedRoleNames() is currently used to construct the array inheritedRoles on a role-assignment, where it only takes over the parameter _id. If you have data stored on the role, wouldn't it be better to have it copied along?

If one of these parameter changes, the programmer extending this package would then be responsible for also keeping this data updated there. If he doesn't need it there, he can extend this package wrapping this function to remove the additional properties.

Guide contributeur

Direction de recherche
Examinez la fonction getInheritedRoleNames() dans roles common.js, comprenez la structure des données de rôle et modifiez la pour copier toutes les propriétés de rôle au lieu de seulement id. Assurez la compatibilité ascendante en fournissant une option pour conserver l'ancien comportement si nécessaire.
Stack technique
javascript
Domaine
backend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60