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

Ajoute une page de preview pour la classification par objectifs #6386

Merged
merged 2 commits into from
Sep 28, 2022

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Sep 5, 2022

Cette PR ajoute une page pour que tout un chacun puisse jeter un œil à la future classification par objectifs, avant qu'une intégration plus aboutie soit préparée. C'est le pendant accessible à tous de la page d'édition en masse réservée au staff.

Au menu :

  • une page avec des filtres pour voir le classement selon les différents objectifs ;
  • un petit message dans la bibliothèque pour que le lien existe quelque part sur le site.

Je suis volontairement resté très simple. Le but est de mettre cette classification aux yeux de tous rapidement ; le travail plus abouti risque d'être long. En conséquence :

  • j'ai mis en commun du code avec la page d'édition en masse pour le staff ;
  • j'ai volontairement peu de tests (la page est simple, vouée à la disparition à plus ou moins brève échéance, et s'appuie essentiellement sur l'édition en masse des objectifs, qui est bien testée).

Contrôle qualité

Vérifier que la nouvelle route marche (en utilisant le lien dans la bibliothèque par exemple).

Vérifier que les liens nous amènent pas sur des pages privées mais bien vers les versions en ligne.

Vérifier qu'il n'y a pas de versions privées qui s'affichent.

@Arnaud-D Arnaud-D added C-Front Concerne l'interface du site C-Back Concerne le back-end Django S-Évolution Ajoute de nouvelles fonctionnalités labels Sep 5, 2022
@coveralls
Copy link

coveralls commented Sep 5, 2022

Coverage Status

Coverage increased (+0.001%) to 88.303% when pulling 31f3212 on Arnaud-D:preview_classif into 8819ef3 on zestedesavoir:dev.

Copy link
Member

@artragis artragis left a comment

Choose a reason for hiding this comment

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

Test OK
mais j'ai deux trois commentaires.

templates/tutorialv2/goals/view-goals.html Outdated Show resolved Hide resolved
templates/tutorialv2/goals/view-goals.html Outdated Show resolved Hide resolved
templates/tutorialv2/view/categories.html Show resolved Hide resolved
@Arnaud-D
Copy link
Contributor Author

J'ai pris en compte les remarques de artragis.

@artragis artragis merged commit 2d9572f into zestedesavoir:dev Sep 28, 2022
@Arnaud-D Arnaud-D deleted the preview_classif branch September 28, 2022 10:22
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 S-Évolution Ajoute de nouvelles fonctionnalités
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants