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

Corrige la mauvaise permission sur les galeries lors de la désinscription #6693

Merged
merged 2 commits into from
Dec 25, 2024

Conversation

philippemilink
Copy link
Member

En regardant #6689), j'ai remarqué que rien ne s'affichait dans la colonne des permissions dans la zone d'admin des galeries pour certaines galeries. Ce sont les galeries des membres qui se sont désinscrits et dont la permission a été changée lors de la désinscription. Cependant, le nouveau droit qui était attribué était w au lieu de W, qui est la valeur déclarée comme valeur possible.

Cette PR corrige le problème pour les futures désinscriptions et applique une migration pour transformer tous les w en W.

Contrôle qualité

  1. Se désinscrire
  2. Constater que rien ne s'affiche dans la colonne "Permissions" de la galerie par défaut du membre désinscrit dans la zone d'admin
  3. Appliquer la migration
  4. Constater que tout est affiché correctement dans la zone d'admin
  5. Désinscrire un autre membre
  6. Constater que tout est affiché correctement dans la zone d'admin, même pour le membre désinscrit juste avant.

@coveralls
Copy link

coveralls commented Dec 15, 2024

Coverage Status

coverage: 89.2%. remained the same
when pulling 21876cc on philippemilink:gallery-update-rights
into 4096ebf on zestedesavoir:dev.

Copy link
Contributor

@Arnaud-D Arnaud-D left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA OK ✔️

@Arnaud-D Arnaud-D enabled auto-merge (squash) December 25, 2024 08:30
@Arnaud-D Arnaud-D merged commit b39c7fc into zestedesavoir:dev Dec 25, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Fusionnée
Development

Successfully merging this pull request may close these issues.

3 participants