refactor(forum): gabarit séparé pour les forums de documentation

[calummackervoy]

Description

🎸 Séparation des gabarits servis selon la type de forum, en partageant le même parent

Type de changement

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

[vincentporte]

Prévois tu d'ajouter des tests de rendus avec snapshot ?

[francoisfreitag]

Je laisse Vincent vérifier les pages, j’ai juste jeté un œil aux tests.
La fixture est très bien. On pourrait imaginer changer la signature pour :

-def reset_model_sequence_fixture(model_class):
+def reset_model_sequence_fixture(*model_classes):

Il est probable qu’on souhaite reset plusieurs séquences. 🤷

[vincentporte]

Tu testes comment jamais la commu n'a été testée ;) Congrats !

[vincentporte]

on a un petit souci sur le rendu des fiches pratiques

actuel

dans la branche en cours

et dans le rendu des thématiques

actuel

dans la branche en cours

A force de nous concentrer sur des details de techs, nous avons failli passer à coté du fonctionnel métier 8-)

[calummackervoy]

@vincentporte t'as une fixture Django avec les thématiques problèmatiques que t'as montré dans l'image que je peux piquer s'il te plaît ? J'ai du mal à réproduire localement :

J'avais bien pensé à tester dans le navigateur et pas juste sur snapshot ! 😄

[calummackervoy]

@vincentporte j'ai ajouté quelques autres tests snapshots pour le contenu des autres gabarits (forum_detail.html et forum_documentation_category.html), et j'ai répondu aux autres retours. Hors de le soucis avec les différentes comportements en locale tout est prêt normalement

[vincentporte]

🎯 LGTM