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

api: add minutes filter on meetings datetimes endpoint (#73832) #62

Merged
ecazenave merged 1 commits from wip/73832-datetimes-minutes into main 2023-03-28 13:29:57 +02:00
Conversation 0 Commits 1 Files Changed 3 +68
ecazenave commented 2023-03-22 15:19:45 +01:00
Owner
Copy Link
No description provided.
ecazenave force-pushed wip/73832-datetimes-minutes from 03b8a16d4d to 18dd03f16e 2023-03-22 15:29:05 +01:00 Compare
ecazenave changed title from WIP: api: add minutes filter on meetings datetimes endpoint (#73832) to api: add minutes filter on meetings datetimes endpoint (#73832) 2023-03-22 15:41:09 +01:00
lguerin reviewed 2023-03-23 08:40:05 +01:00
tests/api/datetimes/test_meetings.py
@ -2611,0 +2650,4 @@
assert datetime_from_str(resp.json['data'][1]['datetime']).minute == 0
# filter on minutes, with more choices
api_url = '/api/agenda/%s/meetings/%s/datetimes/?minutes=0&minutes=30' % (
lguerin commented 2023-03-23 08:40:01 +01:00
Owner
Copy Link

Petite hésitation: c'est utilisable dans wcs ? Le fait d'avoir plusieurs fois le même argument en querystring ?

Petite hésitation: c'est utilisable dans wcs ? Le fait d'avoir plusieurs fois le même argument en querystring ?
ecazenave commented 2023-03-23 16:44:36 +01:00
Author
Owner
Copy Link

Je viens de tester, ça passe dans une source de donnée manuellement configurée et en rentrant l'URL directement en source de donnée sur le champ (dans les deux cas, on rentre l'URL et les paramètres dans un seul champ).

Ça ne passe pas si on duplique une source de donnée agenda et qu'on veut ensuite y ajouter les paramètres. Dans ce cas l'interface sépare url et paramètres et on ne peut pas 'dupliquer' un paramètre.

Perso ça me semble ok.

Je viens de tester, ça passe dans une source de donnée manuellement configurée et en rentrant l'URL directement en source de donnée sur le champ (dans les deux cas, on rentre l'URL et les paramètres dans un seul champ). Ça ne passe pas si on duplique une source de donnée agenda et qu'on veut ensuite y ajouter les paramètres. Dans ce cas l'interface sépare url et paramètres et on ne peut pas 'dupliquer' un paramètre. Perso ça me semble ok.
fpeters commented 2023-03-23 16:46:21 +01:00
Owner
Copy Link

Perso pas; on devrait vouloir encourager l'utilisation de sources de données spécifiques agenda.

Perso pas; on devrait vouloir encourager l'utilisation de sources de données spécifiques agenda.
ecazenave commented 2023-03-23 16:49:00 +01:00
Author
Owner
Copy Link

Et donc on travaille pour permettre de doubler les noms de paramètres dans wcs ou je revoie ma copie ici ?

Et donc on travaille pour permettre de doubler les noms de paramètres dans wcs ou je revoie ma copie ici ?
fpeters commented 2023-03-23 17:13:10 +01:00
Owner
Copy Link

Plutôt pour modifier ici, genre permettre minutes=0,30 ?

Plutôt pour modifier ici, genre permettre minutes=0,30 ?
ecazenave commented 2023-03-28 13:23:14 +02:00
Author
Owner
Copy Link

Voilà.

Voilà.
lguerin marked this conversation as resolved
lguerin approved these changes 2023-03-23 16:46:38 +01:00
ecazenave force-pushed wip/73832-datetimes-minutes from 18dd03f16e to 1827cfe5c6 2023-03-28 13:16:29 +02:00 Compare
ecazenave requested review from lguerin 2023-03-28 13:24:03 +02:00
lguerin approved these changes 2023-03-28 13:25:18 +02:00
ecazenave merged commit 1827cfe5c6 into main 2023-03-28 13:29:57 +02:00
ecazenave deleted branch wip/73832-datetimes-minutes 2023-03-28 13:29:57 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 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#62
No description provided.
Delete Branch "wip/73832-datetimes-minutes"

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?