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

Quelques améliorations pour les objectifs #6510

Merged
merged 3 commits into from
Jun 18, 2023

Conversation

philippemilink
Copy link
Member

Fix #6507

Cette PR fait plusieurs choses :

  • elle utilise les slugs des objectifs dans les URLs qui les concernent au lieu de leur id
  • elle empêche la création d'un objectif ayant pour slug non-classes
  • elle améliore l'interface d'administration des objectifs
  • elle refactorise un peu par endroit le code

Contrôle qualité

  1. Se connecter en tant qu'admin
  2. Aller dans la zone d'administration des objectifs
  3. Ajouter un nouvel objectif. En saisissant le nom de l'objectif le champ pour le slug est automatiquement complété
  4. Essayer de créer un objectif avec pour slug non-classes : une erreur apparaît au moment de la validation
  5. Aller sur la page pour modifier un objectif : en haut à droite, il y a un bouton Voir sur le site qui mène à la page qui liste les contenus ayant cet objectif
  6. Ajouter un objectif à un contenu
  7. Le lien de l'objectif qui apparaît dans l'en-tête du contenu contient le slug de l'objectif et non son ID
  8. Aller sur la page qui liste les contenus ayant cet objectif. Jouer avec les différents filtres, notamment Toutes et Sans objectifs : ils fonctionnent et utilisent des slugs (sauf pour Toutes)
  9. Aller sur la page de modification en masse des objectifs (accessible depuis le menu déroulant du profil) : on arrive sur le page avec le filtre Sans objectif actif. Jouer avec les différents filtres, notamment Toutes et Sans objectifs : ils fonctionnent et utilisent des slugs (sauf pour Toutes)

À celui qui mergera : puisque les commits portent sur des points différents, ne pas squasher les commits.

@coveralls
Copy link

coveralls commented Jun 10, 2023

Coverage Status

coverage: 88.399% (+0.01%) from 88.388% when pulling 610e96d on philippemilink:goals-use-slug into 3460554 on zestedesavoir:dev.

Copy link
Contributor

@Arnaud-D Arnaud-D left a comment

Choose a reason for hiding this comment

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

Globalement ça marche !

Juste :

  • un bug mineur dans l'admin
  • deux remarques mineures sur le code

zds/tutorialv2/views/goals.py Outdated Show resolved Hide resolved
zds/tutorialv2/models/goals.py Outdated Show resolved Hide resolved
Ce slug est utilisé pour afficher les contenus qui n'ont aucun
objectif attribué.
* Ajoute l'auto-complétion du champ pour le slug à partir du nom de
  l'objectif
* Ajoute un lien pour accéder à la page listant les contenus ayant tel
  objectif
@philippemilink
Copy link
Member Author

J'ai corrigé les points que tu as soulevés.

À celui qui mergera : puisque les commits portent sur des points différents, ne pas squasher les commits.

C'est toujours d'actualité :)

@Arnaud-D Arnaud-D enabled auto-merge (rebase) June 18, 2023 09:47
@Arnaud-D Arnaud-D merged commit 5d13328 into zestedesavoir:dev Jun 18, 2023
@philippemilink philippemilink deleted the goals-use-slug branch June 18, 2023 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Les URLs des objectifs devraient contenir leur slug
3 participants