Déplace HelpWriting dans les tutos

[Arnaud-D]

Un des items de #6246.

Déplace le modèle HelpWriting dans les tutos. Quelques subtilités avec les migrations.

Contrôle qualité

Avec la branche de dev (donc avant cette PR) :

• créer un tuto ou article
• mettre des demandes d'aide dessus
• éventuellement faire pareil avec un deuxième
• vérifier que les demandes d'aides sont bien enregistrées en actualisant la page ou en allant sur la page des aides dans la bibliothèque).

Avec la présente PR :

• appliquer les migrations.
• voir que les données sont bien toujours présentes dans la page des aides dans la bibliothèque et sur les tutos avec des demandes d'aide
• tester brièvement que la modification des demandes fonctionne toujours.

[coveralls]

Coverage increased (+0.003%) to 87.994% when pulling 8f03ffb on Arnaud-D:deplace_help_writing into afc088e on zestedesavoir:dev.

[philippemilink]

Il y a avait un conflit pour mettre à jour la PR, je l'ai résolu depuis l'interface de GitHub, mais pas correctement selon le linter. Je te laisse corriger mes erreurs @Arnaud-D ? ;)

[philippemilink]

J'ai un doute sur les deux migrations que tu introduis : j'ai l'impression que ça va supprimer la table correspondante et la recréer ensuite (et donc perdre tout son contenu). J'ai pas encore testé, peut-être que ça fonctionne.

Et puisque tu as bien nommé la deuxième migration, tu pourrais faire de même pour la première :) Et d'ailleurs elles pourraient avoir le même nom, comme ça on saura que ces deux migrations sont liées (quelque chose du style move_help_writing).

[philippemilink]

(Et d'ailleurs la CI ne passe pas à cause des migrations.)

(Étonnant qu'elle soit passée la première fois)

[Arnaud-D]

Bien vu, les migrations ne faisaient pas du tout ce qui était souhaité. Comme ça se passe bien d'habitude, j'ai trop fait confiance à la génération automatique, mais le cas présent est un de ceux qui ne fonctionnent pas, évidemment.

C'est un peu subtil la marche à suivre dans ce cas-ci, vu qu'il faut découpler la base de données et le statut utilisé par l'autodétection, tout en gardant tout bien en ordre. Pour référence, il y a des infos dans la doc Django et surtout je me suis inspiré de cette réponse sur StackOverflow.

À tester attentivement. ^^

[philippemilink]

QA OK ✔️

J'ai aussi vérifié ce que faisaient les migrations avec la commande sqlmigrate :

% py manage.py sqlmigrate utils 0025_move_helpwriting                                                                                                                                                                                                                                                                
BEGIN;
--
-- Custom state/database change combination
--
ALTER TABLE "utils_helpwriting" RENAME TO "tutorialv2_helpwriting";
COMMIT;
% py manage.py sqlmigrate tutorialv2 0033_move_helpwriting                                                                                                                                                                                                                                                           
BEGIN;
--
-- Custom state/database change combination
--
COMMIT;

Donc ça fait bien ce qu'on veut que ça fasse, et uniquement ça ! :)