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

Autorise le staff à voir la biographie d'un membre banni #6500

Merged
merged 2 commits into from
May 27, 2023

Conversation

Migwel
Copy link
Contributor

@Migwel Migwel commented May 23, 2023

Fix #6397, #6204

Contrôle qualité

  • Créez un utilisateur (user1) avec une biographie, une signature et un site web
  • Connectez-vous avec un compte staff et bannissez user1 définitivement

Scénario 1:

  • Restez connecté et allez sur la page de profil de user1
  • Vérifiez que la biographie, la signature et le site web sont toujours visibles

Scénario 2:

  • Déconnectez-vous et allez sur la page de profil de user1
  • Vérifiez que la biographie, la signature et le site web ne sont pas visibles (message: La biographie de ce membre est masquée car il a été banni définitivement.)

Scénario 3:

  • Connectez-vous avec un utilisateur lambda (non-staff)
  • Vérifiez que la biographie, la signature et le site web ne sont pas visibles (message: La biographie de ce membre est masquée car il a été banni définitivement.)

@philippemilink
Copy link
Member

Merci ! Est-ce que tu peux essayer de traiter aussi #6204 dans cette PR ?

@coveralls
Copy link

coveralls commented May 23, 2023

Coverage Status

Coverage: 88.385% (+0.0007%) from 88.385% when pulling 4dd6a27 on Migwel:issue6397 into cf48c6f on zestedesavoir:dev.

@Migwel
Copy link
Contributor Author

Migwel commented May 23, 2023

Merci ! Est-ce que tu peux essayer de traiter aussi #6204 dans cette PR ?

Ah oui, j'avais loupé que l'issue était liée. Ca devrait être simple donc je vais essayer de faire ça

@philippemilink
Copy link
Member

philippemilink commented May 25, 2023

Merci ! Est-ce que tu peux essayer de traiter aussi #6204 dans cette PR ?

Ah oui, j'avais loupé que l'issue était liée. Ca devrait être simple donc je vais essayer de faire ça

Ah mais je viens d'essayer sur la branche dev (pas avec cette PR donc), et je me rends compte que ce qui est demandé dans #6204 est déjà en place. En creusant, je me rends compte que c'est toi qui l'a mis en place avec ta PR #6307. Tu confirmes que cette PR ne concerne pas #6204 puisque c'est déjà en place ?

@Migwel
Copy link
Contributor Author

Migwel commented May 26, 2023

En effet, il y a un peu de confusion dans tout ça.
Tu as raison que #6204 semble déjà être en place. Même en production, qu'on soit staff, simple membre, ou même non-connecté, on peut voir les messages et sujets de membres bannis. Cela dit, je ne pense pas que ça ait été introduit avec #6307, cette issue étant relative au masquage de l'activité de forum des membres qui le souhaitent (sans lien avec les bannis donc).

Dans cette PR-ci, je résous donc uniquement #6397 mais un peu plus que demandé car l'issue ne parle que de biographie mais ma PR affiche aussi la signature et site web / addresse email.

Copy link
Member

@philippemilink philippemilink 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 ✔️

@philippemilink philippemilink enabled auto-merge (rebase) May 27, 2023 18:32
@philippemilink philippemilink enabled auto-merge (squash) May 27, 2023 18:32
@philippemilink philippemilink merged commit ae25244 into zestedesavoir:dev May 27, 2023
Migwel added a commit to Migwel/zds-site that referenced this pull request Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Autoriser le staff à afficher les biographies des membres bannis
3 participants