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

Déplace le choix des catégories dans un formulaire indépendant #5920

Closed
wants to merge 5 commits into from

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Aug 31, 2020

Déplacement de la modification des catégories dans un formulaire dédié sous forme de modale.

Fix #5908.

Contrôle qualité

  • Vérifier l'affichage du bouton d'accès au formulaire (sur les tutos, sur les articles, sur les billets)
  • Vérifier le non-affichage du bouton dans les conditions adéquates (quand on a pas les droits notamment)
  • Vérifier qu'on se prend bien des erreurs de permissions si on essaie d'envoyer le formulaire sans les droits
  • Vérifier que le formulaire marche pour des situations habituelles quand on a les droits
  • Vérifier qu'on ne peut pas enlever toute les catégories d'un contenu publié
  • Jouer un peu à maltraiter le formulaire pour vérifier la robustesse du traitement des erreurs.

@Arnaud-D Arnaud-D added C-Back Concerne le back-end Django C-Front Concerne l'interface du site S-Évolution Ajoute de nouvelles fonctionnalités labels Aug 31, 2020
@Arnaud-D Arnaud-D added this to En développement in Suivi des PR via automation Aug 31, 2020
@Arnaud-D Arnaud-D moved this from En développement to En attente de retours in Suivi des PR Sep 12, 2020
@Arnaud-D Arnaud-D marked this pull request as ready for review September 12, 2020 11:39
@Arnaud-D
Copy link
Contributor Author

Arnaud-D commented Sep 12, 2020

J'ai tenté de modifier un test du front pour qu'il passe, sans succès. Il rate car l'affichage du formulaire aboutit à une modale très longue, ce qui n'est pas top en soi.

J'aimerai bien de l'aide pour trouver un meilleur design (ou corriger le test au pire), parce que je ne m'en sors pas seul.

@artragis , en mai, tu as écrit ça :

Par la suite, je vais réfléchir à un système pour les catégories

T'as des idées concrètes qu'on pourrait intégrer dans cette PR ? :-)

@artragis
Copy link
Member

Perso, je voulais présenter ça sur un élément totalement différent avec un layout plus "moderne" dans le sens où on avait :

  • en premier lieu la catégorie principale, avec la même icône que sur la bibliothèque
  • une fois qu'on a cliqué sur ladite catégorie => un moyen de sélectionner qui prend moins de place (c'est là que je manque d'idée) qu'une liste de cases à cocher

Je voulais vraiment refondre ce système.

@Arnaud-D Arnaud-D force-pushed the formulaire_categories branch 2 times, most recently from 8c6e6ae to 7367ecb Compare October 6, 2020 20:45
@Arnaud-D Arnaud-D force-pushed the formulaire_categories branch 2 times, most recently from 03488c2 to 0d22f69 Compare October 12, 2020 06:45
@Arnaud-D Arnaud-D force-pushed the formulaire_categories branch 3 times, most recently from e138c95 to 0a06e2d Compare December 19, 2020 17:28
@Arnaud-D
Copy link
Contributor Author

Arnaud-D commented Sep 12, 2021

À fusionner ou choisir avec #6097.

@Arnaud-D Arnaud-D moved this from En attente de retours to En attente de reprise in Suivi des PR Sep 12, 2021
@Arnaud-D Arnaud-D added the S-Zombie Ticket ou PR oubliée label Mar 17, 2022
@Situphen
Copy link
Member

Situphen commented Mar 17, 2022

Fermé par décision de la réunion des devs

@Situphen Situphen closed this Mar 17, 2022
Suivi des PR automation moved this from En attente de reprise to Cimetière Mar 17, 2022
@Arnaud-D Arnaud-D deleted the formulaire_categories branch April 28, 2024 06:06
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 S-Zombie Ticket ou PR oubliée
Projects
Archived in project
Suivi des PR
  
Cimetière
Development

Successfully merging this pull request may close these issues.

Déplacer le choix des catégories d'une publication dans un formulaire dédié
3 participants