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

invoicing: a final pool can be deleted (#75558) #43

Merged
lguerin merged 1 commits from wip/75558-delete-final-pool into main 2023-04-11 16:09:31 +02:00
Conversation 1 Commits 1 Files Changed 5 +34 -23
lguerin commented 2023-03-24 11:08:56 +01:00
Owner
Copy Link
No description provided.
lguerin force-pushed wip/75558-delete-final-pool from c1a40f9339 to bd79dcc01a 2023-03-28 10:25:42 +02:00 Compare
pmarillonnet commented 2023-04-11 15:04:03 +02:00
Owner
Copy Link

Je commence à relire.

Je commence à relire.
pmarillonnet reviewed 2023-04-11 15:12:49 +02:00
lingo/invoicing/views.py Outdated
@ -568,3 +566,3 @@
def delete(self, request, *args, **kwargs):
self.object = self.get_object()
if self.object.is_last:
if self.object.is_last and self.object.draft:
pmarillonnet commented 2023-04-11 15:10:56 +02:00
Owner
Copy Link

Pas compris pourquoi on restreint le cas où la campagne devient invalide.
Intuitivement on pourrait se dire qu’à partir du moment où il y a eu suppression, ça invalide la campagne dans tous les cas, que le pool soit un brouillon ou non. Je loupe un truc ?

Pas compris pourquoi on restreint le cas où la campagne devient invalide. Intuitivement on pourrait se dire qu’à partir du moment où il y a eu suppression, ça invalide la campagne dans tous les cas, que le pool soit un brouillon ou non. Je loupe un truc ?
lguerin commented 2023-04-11 15:17:30 +02:00
Author
Owner
Copy Link

On ne peut promouvoir le dernier draft que si la campagne est valide.
Si on supprime le pool définitif, le dernier draft est toujours valide (et la campagne aussi), parce que rien n'a pu changer: ni les pointages, ni les params de date ou d'agenda de la campagne.

En faisant des tests de suppression du pool définitif en local, j'avais alors une campagne invalide et il me fallait relancer un draft pour pouvoir le promouvoir. Et ça n'était pas nécessaire.

On ne peut promouvoir le dernier draft que si la campagne est valide. Si on supprime le pool définitif, le dernier draft est toujours valide (et la campagne aussi), parce que rien n'a pu changer: ni les pointages, ni les params de date ou d'agenda de la campagne. En faisant des tests de suppression du pool définitif en local, j'avais alors une campagne invalide et il me fallait relancer un draft pour pouvoir le promouvoir. Et ça n'était pas nécessaire.
pmarillonnet commented 2023-04-11 15:23:59 +02:00
Owner
Copy Link

Ah oui ok, c’est plus clair, merci.

Ah oui ok, c’est plus clair, merci.
pmarillonnet marked this conversation as resolved
pmarillonnet approved these changes 2023-04-11 15:24:15 +02:00
pmarillonnet left a comment
Owner
Copy Link

Ok

Ok
lguerin changed target branch from wip/75556-invoicing-lock-events-check to main 2023-04-11 16:04:07 +02:00
lguerin added 1 commit 2023-04-11 16:06:08 +02:00
gitea/lingo/pipeline/head This commit looks good Details
b27cff018f
invoicing: a final pool can be deleted (#75558)
lguerin merged commit b27cff018f into main 2023-04-11 16:09:31 +02:00
lguerin deleted branch wip/75558-delete-final-pool 2023-04-11 16:09:31 +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#43
No description provided.
Delete Branch "wip/75558-delete-final-pool"

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?