search

zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir
https://zestedesavoir.com
Other
268 stars 161 forks source link

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

Closed artragis closed 1 month ago

artragis commented 1 year ago

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é.

coveralls commented 2 months ago

Coverage Status

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

Arnaud-D commented 1 month ago

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

philippemilink commented 1 month ago

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
philippemilink commented 1 month ago

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.