-
Notifications
You must be signed in to change notification settings - Fork 1
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
[MODERATION] automatiser la detection des scams #548
Conversation
@francoisfreitag : flaky f…g test : |
b71d001
to
bb59086
Compare
solved |
bb59086
to
d47ef54
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ça m’a l’air de bien fonctionner 👍
Le message de commit « install langdetect python lib (to be rebased after distinct PR to update libs) » peut être mis à jour.
2392153
to
1ab6ccd
Compare
…me for more consistency
984e3ff
to
1283229
Compare
1283229
to
17836bf
Compare
…rated anonymous posts possible again. Only detected scams are automatically disapproved. Temporary warning banners for anonymous posters are deleted"
…eding anonymous_key to get explicitly representative tests
17836bf
to
db343a5
Compare
Je te laisse vérifier “Remove those ugly mock.patch” avant de merger ;) |
tu m'as bien fait marrer 🤣 je t'appelerai Léon maintenant 🔪 |
Description
🎸 Les utilisateurs non authentifiés pourront reprendre le droit
post_without_approval
🎸 Les messages contenant des balises html, écrits dans une langue dlfférente de celle definie dans
LANGUAGE_CODE
ou postés par des utilisateurs dont le nom de domaine est en liste noire sont automatiquement désapprovés.Type de changement
🎢 Nouvelle fonctionnalité (changement non cassant qui ajoute une fonctionnalité).
🚧 technique
Points d'attention
🦺 Le modele
BouncedDomainName
est ajouté🦺 Isolation de la logique business dans
notification.utils.py
🦺 Exploitation de la logique business dans les
CreateView
,UpdateView
classique ethtmx
🦺 fusionner la PR #549 de mise à jour des dépendances auparavant
⚠️
langdetect
est sensible à la longueur du contenu pour déterminer le langage. Pour eviter de rendre les tests fragiles, lescontent
despost
sont hydratés avecfaker.paragraph(nb_sentences=5))
au lieu defaker.text()