Nouveaux ingrédients : options pour demander de l'info et refuser #1303
Conversation
Add fields to serializers, only visible to instructors and visors
hfroot
added
the
CREATION NOUVEL INGREDIENT
| autocomplete_fields = ("plant",) | ||
| extra = 0 | ||
| classes = ["collapse"] |
There was a problem hiding this comment.
je sais pas si c'est voulu ou non, je l'ai ajouté car il me semblait coherent
api/serializers/declaration.py
Outdated
| ) | ||
|
|
||
|
|
||
| class DeclaredPlantSerializer(serializers.ModelSerializer): | ||
| class HideInstructionFields: |
There was a problem hiding this comment.
On a déjà un serializer PrivateCommentSerializer dans api/serializers/utils.py dont l'objectif est d'enlever certains champs en fonction des rôles instruction / visa via to_representation.
Est-ce que ce serait une bonne idée de ajouter les champs request_private_notes et request_status dedans ? (vraie question, je veux bien ton avis).
Si on décidait de le faire, le PrivateCommentSerializer se trouverait à gérer types différentes d'objets, donc la fonction pop devra être accompagnée d'une valeur par défaut pour éviter de gérer les exceptions, par exemple : representation.pop("request_private_notes", None)
There was a problem hiding this comment.
J'ai décidé de rendre les champs à cacher configurable au niveau de l'enfant qui utilise PrivateCommentSerializer (renommé en PrivateFieldsSerializer). J'hesitais refactoriser le fonctionnement de DeclarationSerializer en même temps mais vu que ça touche aussi read_only j'ai décidé de ne pas le faire.
Maquette
Maquette bandeau refusé et info, remplacé par alerte dans cette PR
Suite à cette PR, il y a des changements à faire sur d'autres pages pour afficher les commentaires et le statut des demandes. Ces évolutions sont notés dans les grands issus : #1305 #1307 #1308