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

Tente de créer un raccourcis pour que les modo puissent supprimer les mauvais domaines #6552

Merged
merged 10 commits into from
Oct 17, 2024

Conversation

artragis
Copy link
Member

Je fais une petite PR pour faciliter le ban des domaines de fournisseurs de membres qui ne se connectent que pou spam.
Aujourd'hui une fois le ban fait, il faut aller sur la page des fournisseur, vérifier que le membre qu'on a banni n'est pas en haut de la liste et bannir si c'est nécessaire.

L'ambition (petite) de cette PR est d'ajouter le bouton directement dans l'interface de profile une fois le membre banni.
Mon environnement étant dans les choux, je n'ai pas pu tester, d'où le brouillon, cependant le code devrait être prêt à être reviewé.

templates/member/profile.html Outdated Show resolved Hide resolved
templates/member/profile.html Outdated Show resolved Hide resolved
@Arnaud-D Arnaud-D added C-Front Concerne l'interface du site hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest labels Oct 30, 2023
@coveralls
Copy link

coveralls commented Sep 28, 2024

Coverage Status

coverage: 89.167% (-0.01%) from 89.181%
when pulling 28d7d23 on artragis:feature/quick_domain_ban
into 6918da2 on zestedesavoir:dev.

- corrige le style de la popup
- supprime l'ajout d'un import inutile
- fait la requête pour savoir s'il faut bannir le domaine mail seulement
  si on a le droit de le bannir
@philippemilink philippemilink marked this pull request as ready for review October 1, 2024 21:17
@Arnaud-D
Copy link
Contributor

Arnaud-D commented Oct 16, 2024

À part ma remarque mineure ci-dessus, c'est bon.

@philippemilink
Copy link
Member

Instructions pour la QA :

  1. inscrire un nouvel utilisateur avec un nom de domaine pour l'email pas encore connu du site
  2. activer le compte en utilisant l'adresse envoyée par mail, visible dans la console
  3. se connecter en tant qu'admin, bannir définitivement le membre qui vient de s'inscrire
  4. tester que la popup pour bannir le fournisseur d'email fonctionne bien

Arnaud-D

This comment was marked as outdated.

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 NOK

En fait, j'ai une réserve sur l'affichage. On ne peut pas tester facilement en local avec des droits réalistes. Tous les staff sont admin ou presque sur l'environnement de dév. Je pense que ça s'affiche pour tous les staff, alors que les modérateurs en général n'ont pas la gestion des emails (je ne l'avais pas à l'époque où j'étais staff, de mémoire).

C'est pas bloquant, juste des staff qui auront un bouton qui ne marche pas.

@philippemilink
Copy link
Member

Euh non, puisque provider_to_ban n'est défini que si on le droit de gérer les mails. Si on n'a pas le droit, provider_to_ban est évalué à None/False dans le template, et le bouton ne s'affiche pas.

@Arnaud-D Arnaud-D merged commit 4fb6d14 into zestedesavoir:dev Oct 17, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants