entrouvert
/
lingo
15
0
Fork 0
Code Issues Pull Requests 8 Releases Wiki Activity

invoicing: edit and refresh docket (#88699) #178

Merged
lguerin merged 1 commits from wip/88699-invoicing-docket-refresh into main 2024-04-08 09:58:04 +02:00
Conversation 1 Commits 1 Files Changed 6 +182 -12
lguerin commented 2024-04-02 10:07:40 +02:00
Owner
Copy Link
No description provided.
lguerin added 1 commit 2024-04-02 10:07:41 +02:00
gitea/lingo/pipeline/head This commit looks good Details
646ac0f2fd
invoicing: edit and refresh docket (#88699)
lguerin force-pushed wip/88699-invoicing-docket-refresh from 646ac0f2fd to a68315ad7d 2024-04-02 10:40:09 +02:00 Compare
lguerin changed title from WIP: invoicing: edit and refresh docket (#88699) to invoicing: edit and refresh docket (#88699) 2024-04-02 10:44:40 +02:00
pmarillonnet requested review from pmarillonnet 2024-04-04 11:01:43 +02:00
pmarillonnet commented 2024-04-04 11:01:55 +02:00
Owner
Copy Link

(Je commence à relire.)

(Je commence à relire.)
pmarillonnet reviewed 2024-04-04 11:24:19 +02:00
pmarillonnet left a comment
Owner
Copy Link

Juste deux petits trucs vu au passage, dis-moi ce que tu en penses.

Juste deux petits trucs vu au passage, dis-moi ce que tu en penses.
lingo/invoicing/forms.py Outdated
@ -896,0 +896,4 @@
'date_end': now().date(),
}
else:
self.regie = instance.regie
pmarillonnet commented 2024-04-04 11:20:08 +02:00
Owner
Copy Link

J’ai pas compris ce qui empêche de faire tout le # set initial through data de l’autre embranchement aussi ici et d’avoir seulement dans le if … else la façon dont self.regie est initialisé.

J’ai pas compris ce qui empêche de faire tout le `# set initial through data` de l’autre embranchement aussi ici et d’avoir seulement dans le `if … else` la façon dont `self.regie` est initialisé.
lguerin commented 2024-04-04 16:00:49 +02:00
Author
Owner
Copy Link

si on est en update (=il y a une pk), alors qu'on a un formulaire initialisé avec les données du docket, on ne veut pas les écraser par quelque chose qui se trouve peut-être en GET param

si on est en update (=il y a une pk), alors qu'on a un formulaire initialisé avec les données du docket, on ne veut pas les écraser par quelque chose qui se trouve peut-être en GET param
pmarillonnet commented 2024-04-05 10:25:03 +02:00
Owner
Copy Link

Ah oui bien sûr, ce détail m’avait échappé :)

Ah oui bien sûr, ce détail m’avait échappé :)
pmarillonnet marked this conversation as resolved
lingo/invoicing/forms.py Outdated
@ -908,6 +912,9 @@ class PaymentDocketForm(forms.ModelForm):
)
if filterset.form.is_valid():
payment_queryset = filterset.qs
Payment.objects.filter(regie=self.regie, docket=self.instance).exclude(
pmarillonnet commented 2024-04-04 11:23:03 +02:00
Owner
Copy Link

Pas besoin de gérer une atomicité de ces deux update successifs ?

Pas besoin de gérer une atomicité de ces deux `update` successifs ?
lguerin commented 2024-04-04 16:01:09 +02:00
Author
Owner
Copy Link

ça serait pas mal oui, j'ajoute ça

ça serait pas mal oui, j'ajoute ça
lguerin force-pushed wip/88699-invoicing-docket-refresh from a68315ad7d to 499848bd88 2024-04-04 15:55:57 +02:00 Compare
lguerin added 1 commit 2024-04-04 16:03:13 +02:00
gitea/lingo/pipeline/head This commit looks good Details
5fcb8b3d42
fixup! invoicing: edit and refresh docket (#88699)
pmarillonnet approved these changes 2024-04-05 10:25:24 +02:00
pmarillonnet left a comment
Owner
Copy Link

Ack.

Ack.
lguerin force-pushed wip/88699-invoicing-docket-refresh from 5fcb8b3d42 to f07b816bb7 2024-04-08 08:58:55 +02:00 Compare
lguerin changed target branch from wip/88698-invoicing-docket to main 2024-04-08 09:00:49 +02:00
lguerin merged commit f07b816bb7 into main 2024-04-08 09:58:04 +02:00
lguerin deleted branch wip/88699-invoicing-docket-refresh 2024-04-08 09:58:04 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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#178
No description provided.
Delete Branch "wip/88699-invoicing-docket-refresh"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?