Skip to content
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

Lève des restrictions sur les billets #6441

Closed
wants to merge 11 commits into from

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Dec 23, 2022

Dans le cadre de l'organisation des contenus, cette PR :

  • supprime des fonctions inutilisées en rapport avec la distinction billet/tuto/article ;
  • rend générique certains affichages en utilisant "publication" au lieu de "tutoriel", "article" ou "billet ;
  • déverouille la structure des billets et articles pour permettre tout comme les tutoriels ;
  • permet l'ajout/suppression de contributeurs sur les billets.

Contrôle qualité

  • Créer un profil avec différents tutos, articles, billets, en beta ou non, et voir qu les statistiques sont affichées correctement (pour tester d'éventuelles régressions sur les suppressions de fonctions inutilisées).
  • Vérifier que les affichages rendus génériques fonctionnent bien (voir le diff pour ceux concernés).
  • Jouer à crer un article et billet avec une structure de tutoriel et constater que la publication fonctionne.
  • Tester l'ajout et suppression de contributeurs sur les billets.

@Arnaud-D Arnaud-D added C-Front Concerne l'interface du site C-Back Concerne le back-end Django labels Dec 23, 2022
@Arnaud-D Arnaud-D changed the title Lève dess restrictions sur les billets Lève des restrictions sur les billets Dec 23, 2022
@Arnaud-D Arnaud-D force-pushed the refacto-valide-libre branch 4 times, most recently from df6a963 to bb9fc35 Compare December 24, 2022 09:31
@coveralls
Copy link

coveralls commented Dec 24, 2022

Coverage Status

Coverage: 88.315% (+0.01%) from 88.302% when pulling 2d6c329 on Arnaud-D:refacto-valide-libre into fff03bd on zestedesavoir:dev.

@Arnaud-D Arnaud-D marked this pull request as ready for review December 24, 2022 10:03
Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai créé un billet avec parties, chapitres et sections un peu dans tous les sens, par forcément de façon homogène (ie une partie a un chapitre, une autre partie a directement des sections). Une fois le billet publié, sur la version publique, peu importe le lien sur lequel je clique dans la hiérarchie du contenu (nom de partie, nom de chapitre, nom de section), je me prends une erreur 404. Je pense qu'il y a aussi des changements à faire dans les routes.

EDIT: et bien-sûr, ce serait bien d'ajouter les tests pour détecter ce genre de bugs :)

@Arnaud-D
Copy link
Contributor Author

J'avais effectivement oublié ça. J'ai ajouté les routes manquantes pour les billets et les articles.

J'ai réussi (laborieusement) à ajouter des tests. Je me suis restreint aux billets et articles, et j'ai gardé ça simple. En gros, je crée un contenu avec des conteneurs, je le publie et je regarde qu'on peut voir la version en ligne.

On pourrait enrichir ça encore plus, mais je préfère éviter de faire un excès de zèle à ce niveau.

@Arnaud-D
Copy link
Contributor Author

J'ai ajouté l'affichage des liens vers la partie ou chapitre suivant et précédent. C'était présent que sur les tutoriels, j'ai fait en sorte que ça soit actif partout.

@Arnaud-D
Copy link
Contributor Author

Arnaud-D commented Jan 26, 2023

Ça pourrait être intéressant de fusionner #6457 avant cette PR-ci. #6457 change complètement les templates, la présente PR sera difficilement fusionnable de toute façon ; c'est plus simple pour moi de reprendre cette PR à partir de l'autre que l'inverse.

@philippemilink philippemilink added the En attente de la dépendance Ticket bloqué en attente d'un correctif dans une dépendance utilisée label Jan 26, 2023
@Arnaud-D
Copy link
Contributor Author

Ça va pouvoir reprendre ici puisqu'on a fusionné #6457 !

@Arnaud-D
Copy link
Contributor Author

Comme on peut le voir ci-dessus, j'ai découpé cette PR en plus petits morceaux. Elle est désormais obsolète et je la ferme donc.

@Arnaud-D Arnaud-D closed this Oct 28, 2023
@Arnaud-D Arnaud-D deleted the refacto-valide-libre branch October 28, 2023 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django C-Front Concerne l'interface du site
Projects
Status: Cimetière
Development

Successfully merging this pull request may close these issues.

4 participants