entrouvert
/
chrono
15
1
Fork 0
Code Issues Pull Requests 9 Releases Wiki Activity

Labels Milestones

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 2 months ago
Conversation 1 Commits 2 Files Changed 11
lguerin commented 3 months ago
Owner
There is no content yet.
lguerin added 2 commits 3 months ago
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) 3 months ago
lguerin commented 3 months ago
Poster
Owner

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 3 months ago
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) 3 months ago
lguerin changed target branch from wip/75413-extra-user-block-on-check-page to main 2 months ago
lguerin added 3 commits 2 months ago
gitea/chrono/pipeline/head There was a failure building this commit Details
94c4bb4172
manager: lock check if flag check_locked is enabled (#75416)
lguerin reviewed 2 months ago
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 2 months ago
Poster
Owner

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 %}
lguerin added 1 commit 2 months ago
fpeters approved these changes 2 months ago
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 2 months ago
Owner

c'est plutôt missing check_locked, non ?

c'est plutôt missing check_locked, non ?
lguerin commented 2 months ago
Poster
Owner

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 2 months ago
Owner

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 2 months ago
lguerin merged commit 8892bd7bcb into main 2 months ago
lguerin deleted branch wip/75416-event-lock-check 2 months ago

Reviewers

fpeters approved these changes 2 months ago
gitea/chrono/pipeline/head This commit looks good
Details
The pull request has been merged as 8892bd7bcb.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b wip/75416-event-lock-check main
git pull origin wip/75416-event-lock-check

Step 2:

Merge the changes and update on Gitea.
git checkout main
git merge --no-ff wip/75416-event-lock-check
git push origin main
Sign in to join this conversation.
Reviewers
Request review
No reviewers
fpeters
Labels
Apply labels
Clear labels
No items
No Label
Milestone
Set milestone
Clear milestone
No items
No Milestone
Assignees
Assign users
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
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'wip/75416-event-lock-check'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes