-
Notifications
You must be signed in to change notification settings - Fork 11
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
Fix consolidacion #103
base: master
Are you sure you want to change the base?
Fix consolidacion #103
Conversation
@@ -471,6 +471,10 @@ | |||
# Ver csv_import.py | |||
OPCIONES_CARGAS_TOTALES_COMPLETAS = True | |||
|
|||
# Flag para decidir si usamos el mecanismo de trolling automático. | |||
# Independientemente de este flag, se pueden marcar trolls desde el admin. | |||
CON_ANTITROLLING = True |
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.
en vez de hacer esto, no es posible simplemente comentar "antitrolling" de las INSTALLED_APPS
?
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.
Pero entonces habría que sacar todos los imports de antitrolling
o hacerlos condicionales a que esté la aplicación. ¿Cuál es la contra de esto?
con_error |= (ids_a_procesar & identificaciones_ids) | ||
ids_a_procesar -= identificaciones_ids |
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.
🙌🏽
Este PR agrega un flag que controla si se ejecuta automáticamente el calculador de trolling. Los tests asumen que sí se hace el anti-trolling.
Además simplificamos un poco el código de consolidación.
Closes escrutinio-social-peru#2