Development #39638
création d'une demande ou création d'un brouillon ?
0%
Description
L'action s'appele "création d'une demande", mais vient d'un ticket qui était "initialisation d'un brouillon". Et l'action permet les deux.
Il faudrait peut-être un nom différent valable pour les deux, et que le choix de brouillon ou demande ne soit pas dans les paramètres avancés.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 4 ans
- Lié à Development #33186: Initialisation d'un brouillon ajouté
Mis à jour par Frédéric Péters il y a environ 4 ans
- Bloque Development #39657: création d'une demande, exposer par défaut ajouté
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Statut changé de Nouveau à Information nécessaire
- Assigné à mis à Frédéric Péters
Ou alors je crée deux actions à partir de celle-ci, "Créer une demande" et "Créer un brouillon", des fois c'est plus simple d'être explicite même si ça multiplie un peu les choix dans le menu.
Mis à jour par Thomas Noël il y a environ 4 ans
Mon avis tout personnel : pour moi "Création d'une demande" est assez clair, mais effectivement on ne devrait pas y cacher l'option "Créer en tant que brouillon".
Et je précise, c'est assez clair pour moi, parce que "brouillon" c'est un raccourci pour dire "brouillon de demande"... et parce que je n'ai pas d'alternative à proposer :)
Mis à jour par Stéphane Laget il y a environ 4 ans
Je trouve aussi que "Créer une demande" est assez clair et assez générique.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Assigné à changé de Frédéric Péters à Pierre Cros
Ticket créé avec mon interprétation de #33186#note-88 :
Le comportement par défaut est de créer un brouillon, il m'a fallu un moment pour comprendre pourquoi la demande était pas créée, je souhaiterais qu'on change ça. Ou qu'on change l'intitulé de l'action.
Mis à jour par Thomas Noël il y a environ 4 ans
Frédéric Péters a écrit :
Ticket créé avec mon interprétation de #33186#note-88 :
Le comportement par défaut est de créer un brouillon, il m'a fallu un moment pour comprendre pourquoi la demande était pas créée, je souhaiterais qu'on change ça. Ou qu'on change l'intitulé de l'action.
Je proposerais donc "Créer une demande", avoir une case à cocher "Créer en tant que brouillon" qui ne soit pas cochée par défaut. Et ne pas la cacher.
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Assigné à changé de Pierre Cros à Benjamin Dauvergne
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Fichier 0001-create_formdata-change-default-for-draft-option-3963.patch 0001-create_formdata-change-default-for-draft-option-3963.patch ajouté
- Statut changé de Information nécessaire à Solution proposée
- Patch proposed changé de Non à Oui
J'ai conservé le nom de l'action ("Création d'une demande") parce qu'aucune
autre action n'utilise un verbe à l'infinitif.
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Fichier Firefox_Screenshot_2020-02-11T18-29-25.572Z.png Firefox_Screenshot_2020-02-11T18-29-25.572Z.png ajouté
Ça donne ça.
Mis à jour par Thomas Noël il y a environ 4 ans
Techniquement je valide le patch.
A voir si quelqu'un peut dire "ok" avec la copie d'écran ci-jointe (moi ça me va de mettre l'info "créer en tant que brouillon" tout en haut parce que c'est plutôt important)
... et passer ce ticket en "Solution validée" (ou pas)
Mis à jour par Pierre Cros il y a environ 4 ans
C'est bien, j'ai juste pas envie d'avoir la case à cochée "créer en tant que brouillon" cochée par défaut. Comme l'action s'appelle "Création d'une demande", je préfère que cette case soit décochée par défaut (et si c'est déjà le cas, oubliez-moi).
Au passage, désolé si je pollue un peu le ticket, il faudrait peut-être une info si l'action échoue, j'ai pas testé cet aspect.
Mis à jour par Thomas Noël il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Pierre Cros a écrit :
C'est bien, j'ai juste pas envie d'avoir la case à cochée "créer en tant que brouillon" cochée par défaut. Comme l'action s'appelle "Création d'une demande", je préfère que cette case soit décochée par défaut (et si c'est déjà le cas, oubliez-moi).
Oui elle l'est désormais, (draft = False).
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
Pierre Cros a écrit :
Si quoi échoue ? Un topo sur le comportement actuel :Au passage, désolé si je pollue un peu le ticket, il faudrait peut-être une info si l'action échoue, j'ai pas testé cet aspect.
- si le formdef ciblé n'existe plus ça ne fait rien
- si les mappings sont absents (ou qu'aucun varname ne correspond, voir autre ticket), ça crée quand même la demande/le brouillon, et dans le cas de la demande avec des champs requis vides, il n'y a pas de contrôle sur ce point,
- si un mapping référence un champ qui n'existe plus, ça log une erreur mais ça continue en ignorant ce mapping,
- si une expression foire, idem ça log une erreur (LoggerError), mais ça continue en ignorant ce mapping.
Si vous souhaitez un comportement plus strict, il faut faire un autre ticket et décider si c'est bloquant ou pas pour l'activation de la fonctionnalité (mais l'action sur des champs backoffice a le même comportement, et elle ne log même pas d'erreur dans le cas d'un champ ayant disparu).
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit cf6d818b28a53e56b1a58e4fc4099fd5336ba08b Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Tue Feb 11 19:16:24 2020 +0100 create_formdata: change default for draft option (#39638)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
create_formdata: change default for draft option (#39638)