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

[Classification par objectifs] Édition en masse des objectifs #6371

Merged
merged 5 commits into from
Sep 4, 2022

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Aug 8, 2022

C'est prêt !

L'idée est de réaliser une page qui accélère la gestion de la classification par objectifs, en particulier au démarrage (près de 1400 publications à passer en revue !). Du point de vue de l'utilisateur, c'est une grande liste des publications avec des filtres et des boutons pour activer/désactiver un objectif donné pour chaque contenu. Techniquement, ça envoie des requêtes de manière asynchrone, sur le modèle des demandes d'aides.

Il y a une de la réutilisation du code des demande d'aides pour le front, après une menue refactorisation.

image

Contrôle qualité

Ne pas oublier d'appliquer les migrations.

Non-régression : tester le bon fonctionnement des demandes d'aides

  • créer un contenu
  • jouer avec les demandes d'aides
  • vérifier que les boutons changent bien d'état dans le bon sens
  • vérifier que l'aide est bien mise à jour de manière indépendante (typiquement la page des demandes d'aide)

Avant de passer à la suite : ajuster le paramère "goals_content_per_page" pour voir la pagination.

Fonctionnalité principale :

  • aller sur la page d'édition des objectifs (depuis le menu du profil, lien visible en tant que staff) ;
  • tester les boutons, voir que tout va bien niveau affichage ;
  • voir que les statuts affichés sont corrects (par exemple aller voir le formulaire des objectifs sur la page d'un contenu) ;
  • trafiquer les formulaires pour voir que les erreurs sont bien renvoyées (mais affichées nulle par, c'est un choix) ;
  • tester les filtres.

Permissions à tester :

  • si on n'est pas logué, ça renvoie vers la page de connexion ;
  • si on n'est pas staff, erreur.

@Arnaud-D Arnaud-D added C-Front Concerne l'interface du site C-Back Concerne le back-end Django labels Aug 8, 2022
@coveralls
Copy link

coveralls commented Aug 8, 2022

Coverage Status

Coverage increased (+0.04%) to 88.254% when pulling 2b20a86 on Arnaud-D:mass_edit_goals into 86999da on zestedesavoir:dev.

@Arnaud-D Arnaud-D force-pushed the mass_edit_goals branch 4 times, most recently from 867cf9a to 4204097 Compare August 11, 2022 15:01
@Arnaud-D Arnaud-D marked this pull request as ready for review August 11, 2022 15:02
assets/js/button-toggle.js Outdated Show resolved Hide resolved
assets/js/button-toggle.js Outdated Show resolved Hide resolved
@artragis
Copy link
Member

Sur la béta, j'ai un beau glitch graphique : https://cdn.discordapp.com/attachments/547038766506967041/1010258565925175346/unknown.png

@Arnaud-D
Copy link
Contributor Author

Arnaud-D commented Sep 1, 2022

J'ai corrigé les remarques qui ont été faites.

@artragis
Copy link
Member

artragis commented Sep 4, 2022

c'est opé pour moi ! on peut merger

@artragis artragis merged commit 36c6bbc into zestedesavoir:dev Sep 4, 2022
@Arnaud-D Arnaud-D deleted the mass_edit_goals branch September 5, 2022 06:59
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
None yet
Development

Successfully merging this pull request may close these issues.

3 participants