api: add minutes filter on meetings datetimes endpoint (#73832)
#62
Merged
ecazenave
merged 1 commits from wip/73832-datetimes-minutes into main 2 months ago
lguerin
Loading…
Reference in New Issue
There is no content yet.
Delete Branch 'wip/73832-datetimes-minutes'
Deleting a branch is permanent. It CANNOT be undone. Continue?
03b8a16d4dto18dd03f16e3 months agoWIP: api: add minutes filter on meetings datetimes endpoint (#73832)to api: add minutes filter on meetings datetimes endpoint (#73832) 2 months ago@ -2611,0 +2650,4 @@assert datetime_from_str(resp.json['data'][1]['datetime']).minute == 0# filter on minutes, with more choicesapi_url = '/api/agenda/%s/meetings/%s/datetimes/?minutes=0&minutes=30' % (Petite hésitation: c'est utilisable dans wcs ? Le fait d'avoir plusieurs fois le même argument en querystring ?
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.
Perso pas; on devrait vouloir encourager l'utilisation de sources de données spécifiques agenda.
Et donc on travaille pour permettre de doubler les noms de paramètres dans wcs ou je revoie ma copie ici ?
Plutôt pour modifier ici, genre permettre minutes=0,30 ?
Voilà.
18dd03f16eto1827cfe5c62 months ago1827cfe5c6into main 2 months agoReviewers
1827cfe5c6.Step 1:
From your project repository, check out a new branch and test the changes.Step 2:
Merge the changes and update on Gitea.