panier: pouvoir lier une ligne de facturation à un agenda et à un event (#88544) #175

Merged
lguerin merged 2 commits from wip/88544-basket-activity into main 2024-03-29 08:24:56 +01:00
Owner
No description provided.
lguerin added 2 commits 2024-03-22 18:04:16 +01:00
lguerin force-pushed wip/88544-basket-activity from 25604bf5b0 to ca98ebbbdb 2024-03-25 08:47:58 +01:00 Compare
lguerin changed title from WIP: panier: pouvoir lier une ligne de facturation à un agenda et à un event (#88544) to panier: pouvoir lier une ligne de facturation à un agenda et à un event (#88544) 2024-03-25 08:52:04 +01:00
pmarillonnet requested review from pmarillonnet 2024-03-25 09:50:00 +01:00
Owner

(Je commence à relire.)

(Je commence à relire.)
lguerin changed target branch from wip/88504-basket-information-message to main 2024-03-25 10:06:17 +01:00
lguerin force-pushed wip/88544-basket-activity from ca98ebbbdb to ed493650cf 2024-03-25 10:06:49 +01:00 Compare
pmarillonnet approved these changes 2024-03-25 11:38:45 +01:00
pmarillonnet left a comment
Owner

Ok pour moi, juste un truc vu au passage, pas du tout important et peut-être à laisser de côté, je te laisse voir. Ack.

Ok pour moi, juste un truc vu au passage, pas du tout important et peut-être à laisser de côté, je te laisse voir. Ack.
@ -515,2 +515,4 @@
class BasketLineItemSerializer(serializers.ModelSerializer):
slug = serializers.CharField(required=False, max_length=250)
activity_label = serializers.CharField(required=False, max_length=250)
Owner

Du détail, j’avais le souvenir que pour un ModelSerializer, DRF gérait automatiquement la taille limite des champs en fonction de la définition du modèle (la colonne activity_label du modèle est déjà définie de longueur maximale 250), auquel cas il n’y aurait pas besoin de la redéfinir ici.

Du détail, j’avais le souvenir que pour un ModelSerializer, DRF gérait automatiquement la taille limite des champs en fonction de la définition du modèle (la colonne activity_label du modèle est déjà définie de longueur maximale 250), auquel cas il n’y aurait pas besoin de la redéfinir ici.
Author
Owner

c'était pour avoir un champ not required :)

c'était pour avoir un champ not required :)
lguerin merged commit 1c2a38b19d into main 2024-03-29 08:24:56 +01:00
lguerin deleted branch wip/88544-basket-activity 2024-03-29 08:24:56 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/lingo#175
No description provided.