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

Facturation: verrouillage auto des pointages (#75556) #42

Merged
lguerin merged 3 commits from wip/75556-invoicing-lock-events-check into main 2023-04-11 16:04:55 +02:00
Conversation 2 Commits 3 Files Changed 11 +496 -11
lguerin commented 2023-03-24 10:29:03 +01:00
Owner
Copy Link
No description provided.
lguerin commented 2023-03-24 10:31:20 +01:00
Author
Owner
Copy Link

0001: appeler chrono au lancement d'un pool pour lock les pointages
0002: un bouton pour unlock les pointages
0003: un mécanisme pour invalider une campagne

Plutôt que d'invalider le dernier pool, j'invalide la campagne. En pratique c'est la même chose mais pour les queryset c'est plus simple à écrire.

Une campagne devient invalide lorsque:

  • on supprime le dernier pool
  • on unlock les pointages
    Une campagne devient valide lorsqu'on lance un pool.

On ne peut pas promouvoir un pool lorsque la campagne est invalide.
Je n'affiche pas le bouton "unlock" lorsque la campagne est invalide.

0001: appeler chrono au lancement d'un pool pour lock les pointages 0002: un bouton pour unlock les pointages 0003: un mécanisme pour invalider une campagne Plutôt que d'invalider le dernier pool, j'invalide la campagne. En pratique c'est la même chose mais pour les queryset c'est plus simple à écrire. Une campagne devient invalide lorsque: - on supprime le dernier pool - on unlock les pointages Une campagne devient valide lorsqu'on lance un pool. On ne peut pas promouvoir un pool lorsque la campagne est invalide. Je n'affiche pas le bouton "unlock" lorsque la campagne est invalide.
lguerin force-pushed wip/75556-invoicing-lock-events-check from 4f848ffaa7 to 815b47950c 2023-03-30 15:58:36 +02:00 Compare
lguerin changed target branch from wip/75553-campaign-agendas to main 2023-04-03 15:12:45 +02:00
lguerin added 3 commits 2023-04-03 15:14:12 +02:00
gitea/lingo/pipeline/head This commit looks good Details
085433f1f4
invoicing: invalidate a campaign (#75556)
pmarillonnet commented 2023-04-04 15:20:37 +02:00
Owner
Copy Link

Je commence à relire.

Je commence à relire.
pmarillonnet reviewed 2023-04-11 15:00:52 +02:00
pmarillonnet left a comment
Owner
Copy Link

Juste deux petites remarques prises au passage, dis-moi ce que tu en penses.

Juste deux petites remarques prises au passage, dis-moi ce que tu en penses.
lingo/invoicing/templates/lingo/invoicing/manager_campaign_detail.html
@ -70,6 +70,9 @@
{% if not has_running_pool and not has_real_pool %}
<div class="panel--buttons">
<a class="pk-button" rel="popup" href="{% url 'lingo-manager-invoicing-pool-add' regie_pk=regie.pk pk=object.pk %}">{% trans 'Start a pool' %}</a>
{% if not object.invalid %}
pmarillonnet commented 2023-04-06 12:40:21 +02:00
Owner
Copy Link

Question d’UI, peut-être juste ici indiquer que le déverrouillage n’est pas possible car la campagne est invalide ? Je sens déjà venir les tickets clients “Le bouton de déverrouillage a disparu !!” :)

Question d’UI, peut-être juste ici indiquer que le déverrouillage n’est pas possible car la campagne est invalide ? Je sens déjà venir les tickets clients “Le bouton de déverrouillage a disparu !!” :)
lguerin commented 2023-04-11 15:10:03 +02:00
Author
Owner
Copy Link

On a déjà un warning en haut de page qui indique qu'il faut relancer un pool, je pense que ça suffit, non ?

On a déjà un warning en haut de page qui indique qu'il faut relancer un pool, je pense que ça suffit, non ?
pmarillonnet commented 2023-04-11 15:15:56 +02:00
Owner
Copy Link

Ah oui ok j’avais oublié cela, merci.

Ah oui ok j’avais oublié cela, merci.
pmarillonnet marked this conversation as resolved
lingo/invoicing/views.py
@ -521,6 +567,8 @@ class PoolDeleteView(DeleteView):
def delete(self, request, *args, **kwargs):
self.object = self.get_object()
if self.object.is_last:
pmarillonnet commented 2023-04-11 14:59:04 +02:00
Owner
Copy Link

Pareil ici niveau interface, est-ce qu’il y a l’info qui remonte sur le fait que la campagne est invalide parce que le dernier pool a été supprimé ?

Pareil ici niveau interface, est-ce qu’il y a l’info qui remonte sur le fait que la campagne est invalide parce que le dernier pool a été supprimé ?
lguerin commented 2023-04-11 15:10:20 +02:00
Author
Owner
Copy Link

oui, un warning :)

oui, un warning :)
pmarillonnet marked this conversation as resolved
lguerin reviewed 2023-04-11 15:10:33 +02:00
lingo/invoicing/views.py
@ -285,2 +286,4 @@
kwargs['has_running_pool'] = any(p.status in ['registered', 'running'] for p in kwargs['pools'])
kwargs['has_real_pool'] = any(not p.draft for p in kwargs['pools'])
if self.object.invalid:
messages.warning(self.request, _('The last pool is invalid, please start a new pool.'))
lguerin commented 2023-04-11 15:10:33 +02:00
Author
Owner
Copy Link

ici le warning

ici le warning
pmarillonnet commented 2023-04-11 15:16:36 +02:00
Owner
Copy Link

Nickel.

Nickel.
pmarillonnet marked this conversation as resolved
pmarillonnet approved these changes 2023-04-11 15:16:54 +02:00
pmarillonnet left a comment
Owner
Copy Link

Ok pour moi.

Ok pour moi.
lguerin merged commit 9287ea94c5 into main 2023-04-11 16:04:55 +02:00
lguerin deleted branch wip/75556-invoicing-lock-events-check 2023-04-11 16:04:55 +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#42
No description provided.
Delete Branch "wip/75556-invoicing-lock-events-check"

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?