feat(forum_moderation): enregistrement des messages bloquées #659

calummackervoy · 2024-06-06T12:58:16Z

Description

🎸 Enregistrer les messages bloqués lors de la soumission des Topic et des Post
🎸 Enregistrer les messages supprimés lors de la validation de la désapprobation

Type de changement

🎢 Nouvelle fonctionnalité (changement non cassant qui ajoute une fonctionnalité).

Points d'attention

🦺 Migrations inclues

Captures d'écran (optionnel)

lacommunaute/forum_conversation/models.py

lacommunaute/forum_moderation/utils.py

vincentporte · 2024-06-06T14:04:34Z

Le titre de la PR est repris dans le changelog. J'ai l'habitude d'ajouter le nom de l'application ou de la fonctionnalité avant les : du nom. Ici ce pourrait être feat(forum_conversation): xxxx
En français, nous disons plutôt enregistrement que enregistration ;)

vincentporte

Ton code me parait très propre 👏🏿

lacommunaute/forum_conversation/models.py

lacommunaute/forum_conversation/migrations/0010_blockedpost_block_reason.py

lacommunaute/forum_conversation/forms.py

lacommunaute/forum_conversation/tests/tests_views_htmx.py

locale/fr/LC_MESSAGES/django.po

lacommunaute/forum_moderation/views.py

vincentporte

Quelques petites bricoles, sinon, c'est ok pour moi

lacommunaute/forum_conversation/migrations/0009_alter_certifiedpost_options.py

lacommunaute/forum_conversation/models.py

vincentporte · 2024-06-10T07:23:37Z

lacommunaute/forum_moderation/models.py

+    )
+    username = models.EmailField(blank=True, null=True, verbose_name=("Adresse email"))
+    content = models.CharField(verbose_name=_("Content"))
+    block_reason = models.CharField(


"pinaillage" (nitpicking) : block_reason n'a pas besoins d'être blank=True, null=True

locale/fr/LC_MESSAGES/django.po

francoisfreitag

🎯

lacommunaute/forum_moderation/models.py

calummackervoy · 2024-06-11T08:17:51Z

@francoisfreitag @vincentporte sur les retours null=True, blank=True, pour les champs username et block_reason je préférais les inclure - afin d'être consistent avec les champs correspondents de django-machina (et de résoudre les erreurs liés dans nos tests) - si ça vous semble propre ?

vincentporte · 2024-06-11T09:00:21Z

@francoisfreitag @vincentporte sur les retours null=True, blank=True, pour les champs username et block_reason je préférais les inclure - afin d'être consistent avec les champs correspondents de django-machina (et de résoudre les erreurs liés dans nos tests) - si ça vous semble propre ?

Pour username, je suis ok avec toi. D'autant que la règle est "soit le poster, soit le username doit être renseigné, selon que l'utilisateur est authentifié ou non"
Pour block_reason, ce serait dommage d'enregistrer des messages bloqués sans savoir pourquoi. Mais rien de dramatique non plus

calummackervoy · 2024-06-11T12:34:07Z

OK merci ! J'ai mis à jour le PR avec l'username nullable. En pratique lorsque on crée le BlockedPost on met à jour le update_reason, donc en réflexion je l'ai laissé non-nullable pour que on l'enforce jusqu'à on a une raison pour modifier le règle explicitement

francoisfreitag · 2024-06-11T12:42:54Z

lacommunaute/forum_moderation/models.py

+    )
+    username = models.EmailField(null=True, blank=True, verbose_name=("Adresse email"))
+    content = models.CharField(verbose_name=_("Content"))
+    block_reason = models.CharField(verbose_name=_("Block Reason"))


Peut-être également préciser que choices=BlockedPostReason ?

J'ai remplacé le Enum avec models.TextChoices pour être plus consistent avec la reste de la base du code : 9b46db4

…ble BlockedPost

… validation de la désapprobation

nomanculture de l'application

vincentporte

🎯 LGTM

🤖 I have created a release *beep* *boop* --- ## [2.10.0](v2.9.2...v2.10.0) (2024-06-13) ### Features * **admin:** ajoute PostInline à TopicAdmin ([#657](#657)) ([6f3049a](6f3049a)) * ajout du chemin de fer dans l'espace d'échanges et la documentation ([#649](#649)) ([7cc5d14](7cc5d14)) * **forum_conversation/admin:** CertifiedPost verbose_name et traduction ([#666](#666)) ([e9b0da6](e9b0da6)) * **forum_conversation/tests.py:** test explicite pour la seconde page ([#669](#669)) ([5bea4f1](5bea4f1)) * **forum_conversation:** remettre les annonces dans la liste des messages par défaut ([#662](#662)) ([bb341cb](bb341cb)) * **forum_moderation:** enregistrement des messages bloquées ([#659](#659)) ([7318b5d](7318b5d)) * **forum:** harmoniser la liste des topics ([#665](#665)) ([6f8bf3c](6f8bf3c)) * **stats:** afficher les stats quotidiennes des Diagnostics Parcours IAE ([#660](#660)) ([5c74c56](5c74c56)) * **stats:** ajout de la vue historique des visiteurs mensuels ([#656](#656)) ([804be3c](804be3c)) * **stats:** collecter le nombre de diag parcours iae réalisés quotidiennement ([#658](#658)) ([7205a2b](7205a2b)) ### Bug Fixes * **documentation:** améliorer l'affichage des bannières des fiches pratiques ([#653](#653)) ([3b539d2](3b539d2)) * **forum_conversation:** page 1 forcée dans les liens tags ([#668](#668)) ([a150ab2](a150ab2)) * **forum_moderation:** motif de blocage illisible dans l'admin ([#672](#672)) ([5480394](5480394)) * **quality:** formatage des balises `script` dans le gabarit `base.html` ([#651](#651)) ([0be3c8b](0be3c8b)) * **stats:** corrections mineures ([#661](#661)) ([d3ae4fd](d3ae4fd)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

calummackervoy requested a review from vincentporte June 6, 2024 12:58

calummackervoy self-assigned this Jun 6, 2024

calummackervoy linked an issue Jun 6, 2024 that may be closed by this pull request

[SPAMMERS] enregistrer les messages bloqués pour analyse et traçabilité #613

Closed

calummackervoy commented Jun 6, 2024

View reviewed changes

lacommunaute/forum_conversation/models.py Outdated Show resolved Hide resolved

calummackervoy commented Jun 6, 2024

View reviewed changes

lacommunaute/forum_moderation/utils.py Outdated Show resolved Hide resolved

vincentporte reviewed Jun 6, 2024

View reviewed changes

vincentporte requested a review from francoisfreitag June 6, 2024 15:30

calummackervoy changed the title ~~feat: enregistration des messages bloquées~~ feat(forum_moderation): enregistrement des messages bloquées Jun 6, 2024

calummackervoy force-pushed the 613-enregistration-messages-bloquees branch from 519e4e6 to deae2d2 Compare June 7, 2024 11:08

calummackervoy requested a review from vincentporte June 7, 2024 11:08

vincentporte reviewed Jun 10, 2024

View reviewed changes

francoisfreitag reviewed Jun 10, 2024

View reviewed changes

lacommunaute/forum_moderation/models.py Outdated Show resolved Hide resolved

calummackervoy force-pushed the 613-enregistration-messages-bloquees branch from bb89b36 to eb9ab62 Compare June 11, 2024 08:05

calummackervoy force-pushed the 613-enregistration-messages-bloquees branch from 9897a23 to 6664fe5 Compare June 11, 2024 12:17

francoisfreitag approved these changes Jun 11, 2024

View reviewed changes

calummackervoy added 11 commits June 11, 2024 17:27

feat(forum_conversation): ajoute BlockedPost modèle

a958894

feat(forum_conversation): sauvegarde les messages bloquées dans le ta…

29e60f9

…ble BlockedPost

feat(forum_conversation): BlockedPost admin

089036f

update(forum_conversation): traductions des modèles

efe98b6

fix(search): traduction française manquante

7e863cd

fix(build): migrations manquantes

3dc9f24

feat(forum_moderation): enregistrer les messages supprimés lors de la…

548633b

… validation de la désapprobation

refactor(forum): migration BlockedPost à forum_moderation

c5953fb

refactor(forum_moderation): déplacement d'enum dans un fichier enums.py

25ea3cf

style(forum_conversation): suppression du commentaire

d466965

refactor(tests du forum): assertions succinctes

c9b11e7

calummackervoy added 11 commits June 11, 2024 17:27

feat(forum_conversation): nom du CertifiedPost traduit

8c9b100

fix(locale): Postes -> Message

63d22e0

nomanculture de l'application

refactor(PostDisapproveView): création du BlockedPost dans post()

0565328

feat(BlockedPost): utilisateurs authentifié traqués sur le modèle

826096f

refactor(BlockedPost): conversion explicite à CharField

1180ce1

style(BlockedPost): enlève paramètres superflus de block_reason

493d00b

style(BlockedPost): enlève paramètres superflus d'username

5d6903b

style: enlève changements du CertifiedPost pour plus de clarité

06e1fe6

fix(BlockedPost): remet username nullable

58b69d0

fix(CertifiedPost): missing changes from origin

8b5ce01

refactor(BlockedPostReason): remplace Enum avec models.TextChoices

9b46db4

calummackervoy force-pushed the 613-enregistration-messages-bloquees branch from 2d89929 to 9b46db4 Compare June 11, 2024 15:27

calummackervoy requested a review from francoisfreitag June 11, 2024 15:29

vincentporte self-requested a review June 11, 2024 16:03

vincentporte approved these changes Jun 11, 2024

View reviewed changes

calummackervoy merged commit 7318b5d into master Jun 12, 2024
8 checks passed

calummackervoy deleted the 613-enregistration-messages-bloquees branch June 12, 2024 06:57

github-actions bot mentioned this pull request Jun 11, 2024

chore(master): release 2.10.0 #652

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(forum_moderation): enregistrement des messages bloquées #659

feat(forum_moderation): enregistrement des messages bloquées #659

calummackervoy commented Jun 6, 2024

vincentporte commented Jun 6, 2024

vincentporte left a comment

vincentporte left a comment

vincentporte Jun 10, 2024

francoisfreitag left a comment

calummackervoy commented Jun 11, 2024

vincentporte commented Jun 11, 2024

calummackervoy commented Jun 11, 2024

francoisfreitag Jun 11, 2024

calummackervoy Jun 11, 2024

vincentporte left a comment

feat(forum_moderation): enregistrement des messages bloquées #659

feat(forum_moderation): enregistrement des messages bloquées #659

Conversation

calummackervoy commented Jun 6, 2024

Description

Type de changement

Points d'attention

Captures d'écran (optionnel)

vincentporte commented Jun 6, 2024

vincentporte left a comment

Choose a reason for hiding this comment

vincentporte left a comment

Choose a reason for hiding this comment

vincentporte Jun 10, 2024

Choose a reason for hiding this comment

francoisfreitag left a comment

Choose a reason for hiding this comment

calummackervoy commented Jun 11, 2024

vincentporte commented Jun 11, 2024

calummackervoy commented Jun 11, 2024

francoisfreitag Jun 11, 2024

Choose a reason for hiding this comment

calummackervoy Jun 11, 2024

Choose a reason for hiding this comment

vincentporte left a comment

Choose a reason for hiding this comment