entrouvert
/
chrono
14
1
Fork 0
Code Issues Pull Requests 8 Releases Wiki Activity

Facturation: avoir un flag sur Event pour noter que l'événement est verrouillé (#75416) #56

Merged
lguerin merged 2 commits from wip/75416-event-lock-check into main 2023-04-03 16:27:27 +02:00
Conversation 1 Commits 2 Files Changed 11 +462 -31
lguerin commented 2023-03-17 10:41:45 +01:00
Owner
Copy Link
No description provided.
lguerin changed title from Facturation: avoir un flag sur Event pour noter que l'événement est verrouillé (#75416) to WIP: Facturation: avoir un flag sur Event pour noter que l'événement est verrouillé (#75416) 2023-03-17 11:02:02 +01:00
lguerin commented 2023-03-17 11:05:15 +01:00
Author
Owner
Copy Link

Partie API à reprendre: lingo appellera chrono avec en param une date de début, une date de fin, une liste d'agendas.
=> nouvel endpoint

Chrono posera le flag ckeck_locked sur les events concernés.

Partie API à reprendre: lingo appellera chrono avec en param une date de début, une date de fin, une liste d'agendas. => nouvel endpoint Chrono posera le flag `ckeck_locked` sur les events concernés.
lguerin force-pushed wip/75416-event-lock-check from 45efc2e7fa to 141ff5b038 2023-03-17 15:22:23 +01:00 Compare
lguerin changed title from WIP: Facturation: avoir un flag sur Event pour noter que l'événement est verrouillé (#75416) to Facturation: avoir un flag sur Event pour noter que l'événement est verrouillé (#75416) 2023-03-17 15:23:29 +01:00
lguerin changed target branch from wip/75413-extra-user-block-on-check-page to main 2023-03-30 11:21:54 +02:00
lguerin added 1 commit 2023-03-30 11:46:28 +02:00
lguerin reviewed 2023-03-30 11:54:42 +02:00
chrono/manager/templates/chrono/manager_event_check_booking_fragment.html Outdated
@ -35,3 +35,2 @@
</td>
{% if not event.checked or not agenda.disable_check_update %}
<td class="booking-actions" data-{{ booking.kind }}-id="{{ booking.id }}">
{% if not event.check_locked %}
lguerin commented 2023-03-30 11:54:42 +02:00
Author
Owner
Copy Link

le diff est vraiment très moche, en pratique c'est juste un ajout de {% if not event.check_locked %}...{% endif %}

le diff est vraiment très moche, en pratique c'est juste un ajout de {% if not event.check_locked %}...{% endif %}
fpeters approved these changes 2023-04-03 16:12:17 +02:00
tests/api/test_event.py Outdated
@ -1598,0 +1616,4 @@
def test_events_check_lock_params(app, user):
app.authorization = ('Basic', ('john.doe', 'password'))
# missing user_external_id
fpeters commented 2023-04-03 16:08:02 +02:00
Owner
Copy Link

c'est plutôt missing check_locked, non ?

c'est plutôt missing check_locked, non ?
lguerin commented 2023-04-03 16:18:20 +02:00
Author
Owner
Copy Link

yes

yes
lguerin marked this conversation as resolved
tests/api/test_event.py Outdated
@ -1598,0 +1641,4 @@
'/api/agendas/events/check-lock/',
params={
'check_locked': True,
'agendas': 'foo, bar',
fpeters commented 2023-04-03 16:11:31 +02:00
Owner
Copy Link

Je comprends que c'est parce qu'on partage du code (AgendaSlugsMixin) avec des bouts appelés depuis w.c.s. mais je note que peut-être là-dedans un jour on pourrait imaginer une évolution pour également accepter une liste de chaines.

Je comprends que c'est parce qu'on partage du code (AgendaSlugsMixin) avec des bouts appelés depuis w.c.s. mais je note que peut-être là-dedans un jour on pourrait imaginer une évolution pour également accepter une liste de chaines.
lguerin added 2 commits 2023-04-03 16:17:45 +02:00
lguerin merged commit 8892bd7bcb into main 2023-04-03 16:27:27 +02:00
lguerin deleted branch wip/75416-event-lock-check 2023-04-03 16:27:27 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
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/chrono#56
No description provided.
Delete Branch "wip/75416-event-lock-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?