Meteor-Community-Packages/meteor-roles

Migration Script should delete old roles

Open

#300 geöffnet am 28. Nov. 2019

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (919 Stars) (161 Forks)batch import
help wantedv1

Beschreibung

The migration scripts do not delete the old roles in the Roles collection:

{ "_id" : "admin", "children" : [ ] } //new role is created
{ "_id" : "u9jJZoAfaEApp6QBw", "name" : "admin" } // old role is still here

Shouldn't the old roles go the way of the dodo?

Contributor Guide

Tech Stack
javascript
Domain
backenddatabase
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
MeteorMongoDBmigration patterns
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40
Research-Richtung
The issue points out that migration scripts in the meteor roles package do not delete old role documents, leaving duplicates. To fix, locate the migration script files (likely in a 'migrations' or 'server' directory), add logic to remove documents from the Roles collection that have an 'name' field matching the new role's name, or use a unique constraint. Check existing PRs or comments for more context, and ensure backward compatibility.