Envoyer message depuis la page de validation de demande #619
Conversation
liviaribeiro
changed the title
liviaribeiro
changed the title
| @@ -384,7 +384,7 @@ def get_success_url(self): | |||
|
|
|||
| def form_valid(self, form): | |||
| message: RequestMessage = form.save(commit=False) | |||
| message.sender = MessageStakeholders.ISSUER.value | |||
| message.sender = MessageStakeholders.ISSUER.name | |||
There was a problem hiding this comment.
Il y avait un bug d'enregistrement de message. Le "sender" était vide. Ça marchait dans le front car on fait {% if message.sender == "AC" %} {% else %}
There was a problem hiding this comment.
Je suis toujours un peu perdue entre les value et les name sur ces constantes, moi aussi.
There was a problem hiding this comment.
pareil du coup. a chaque fois je me plante, à chaque fois :)
There was a problem hiding this comment.
Ça me rassure beaucoup de vous lire ^^
| @@ -373,6 +374,10 @@ class ValidationForm(PatchedForm): | |||
| "à habiliter ne sont pas des élus.", | |||
| ) | |||
|
|
|||
| content = CharField( | |||
There was a problem hiding this comment.
Je l'appellerais plutôt "message" ou "message_content" pour que ce soit plus explicite à la relecture.
| @@ -15,32 +15,30 @@ <h2>Récapitulatif de la demande</h2> | |||
| Il est possible de les modifier si besoin. | |||
| </p> | |||
| {% include "_display_org_request.html" with organisation=organisation show_all_buttons=True %} | |||
| <!-- | |||
| <br> | |||
There was a problem hiding this comment.
Normalement il n'y a pas besoin de <br> ici, la marge du h3 en-dessous devrait suffire pour être conforme à la charte, non ?
There was a problem hiding this comment.
D'après la spec HTML, les
sont à utiliser seulement pour les retours à la ligne qui font vraiment partie du contenu, pas pour la mise en page :
https://www.w3.org/TR/2014/REC-html5-20141028/text-level-semantics.html#the-br-element
There was a problem hiding this comment.
En fait quand j’ai changé la position du <form> j’ai perdu l’espace qu’il y avait. J’ai donc fait cette magouille pour corriger rapidement 😬
There was a problem hiding this comment.
Ah oui c'est parce que je supprime la marge du dessus quand le h3 est le premier enfant de son parent… Une règle valable pour partout ailleurs u_u
Idées de solutions :
- sortir le h3 du form
- faire un form qui englobe plus de choses
- laisser le form autour du bloc d'en bas, et utiliser l'attribut form sur le nouveau textarea
liviaribeiro
force-pushed
the
lpr/messages-validation-view
branch
4 times, most recently
from
05a2d53
to
fca4fa9
Compare
liviaribeiro
force-pushed
the
lpr/messages-validation-view
branch
from
fca4fa9
to
62b8665
Compare
🌮 Objectif
Pouvoir envoyer un message au moment de la validation de demande d'habilitation.
🔍 Implémentation
Ajout du champ content dans ValidationForm et création de message dans la méthode save(). Mise à jour de validation_form.html. Mise à jour du test pour vérifier la création du message.
🏕 Amélioration continue
(optionnel) Documentation, commandes à lancer, variables d'environment, etc
🖼️ Images
(optionnel) Une ou plusieurs captures d'écran