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

hide_forum_activity doit aussi s'appliquer à l'entête du profil #6504

Merged
merged 2 commits into from
May 27, 2023

Conversation

Migwel
Copy link
Contributor

@Migwel Migwel commented May 24, 2023

Fix #6449

Contrôle qualité

  • Créer un utilisateur avec des posts et messages sur le forum
  • Dans le profil de cet utilisateur, cocher l'option "Masquer mes activités de forum et commentaires sur mon profil (sauf pour moi et l'équipe de modération)"

Avec cet utilisateur:

  • Se rendre sur sa page de profil et vérifier que les liens relatifs à l'activité du forum (messages, sujets créés, sujets suivis) sont visibles

Avec un utilisateur staff:

  • Se rendre sur la page de profil de l'utilisateur créé plus haut et vérifier que les liens relatifs à l'activité du forum (messages, sujets créés) sont visibles

Avec un utilisateur lambda / En étant déconnecté:

  • Se rendre sur sa page de profil et vérifier que les liens relatifs à l'activité du forum ne sont pas visibles

Dans chacun de ces cas, l'activité relative aux publications (tutoriels, articles et billets créés) devrait toujours être visible

@coveralls
Copy link

coveralls commented May 24, 2023

Coverage Status

Coverage: 88.387% (+0.002%) from 88.385% when pulling 4214206 on Migwel:issue6449 into ae25244 on zestedesavoir:dev.

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.

Ça fonctionne, mais un détail me gène. Lorsque le membre choisit de masquer son activité sur les forums, il est affiché dans l'en-tête du profil "Aucun message" (lorsqu'on n'a pas les droits pour voir cette information). Plutôt que de n'afficher "Aucun message", je n'afficherai tout simplement rien : ça laisse penser que le membre n'a vraiment posté aucun message, alors qu'en vrai il a juste désactivé l'affichage de son activité.

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 (squash) May 27, 2023 18:50
@philippemilink philippemilink merged commit 8cc4218 into zestedesavoir:dev May 27, 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.

En-tête profil ne prend pas en compte le param activités forum
3 participants