entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 15 Releases Activity

utils: add date type to endpoint parameters (#72641) #15

Closed
nroche wants to merge 1 commits from wip/72641-date-type-parameter into main
pull from: wip/72641-date-type-parameter
merge into: entrouvert:main
Conversation 1 Commits 1 Files Changed 8 +92 -78
nroche commented 2022-12-20 12:09:18 +01:00
Owner
Copy Link

Proposition d'ajouter un type date sur les paramètres des endpoints
parce que j'ai l'impression que c'est récurrent.
(par exemple, ça pourrait être utilisé pour #72563)

Après, comme on ne peut pas passer de date par défaut (et aussi
que l'on doit spécifier le type si l'on passe une valeur d'exemple),
ça sort du cadre actuel, ce qui explique peut-être pourquoi ce n'a
pas été fait et qui serait une raison pour rejeter ce ticket.

Proposition d'ajouter un type date sur les paramètres des endpoints parce que j'ai l'impression que c'est récurrent. (par exemple, ça pourrait être utilisé pour #72563) Après, comme on ne peut pas passer de date par défaut (et aussi que l'on doit spécifier le type si l'on passe une valeur d'exemple), ça sort du cadre actuel, ce qui explique peut-être pourquoi ce n'a pas été fait et qui serait une raison pour rejeter ce ticket.
nroche force-pushed wip/72641-date-type-parameter from 57c16e3072 to 05b284e982 2022-12-20 12:40:38 +01:00 Compare
bdauvergne approved these changes 2022-12-20 15:47:52 +01:00
bdauvergne left a comment
Owner
Copy Link

Ça me parait bien. À pousser vendredi.

Ça me parait bien. À pousser vendredi.
fpeters commented 2023-01-08 13:18:37 +01:00
Owner
Copy Link

Ça s'est trouvé mergé mais pas via gitea.

Ça s'est trouvé mergé mais pas via gitea.
fpeters closed this pull request 2023-01-08 13:18:37 +01:00
Some checks reported errors
gitea-wip/passerelle/pipeline/pr-main This commit looks good
Details
gitea-wip/passerelle/pipeline/head Build started...
Details
gitea/passerelle/pipeline/head Something is wrong with the build of this commit
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
bdauvergne
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/passerelle#15
No description provided.
Delete Branch "wip/72641-date-type-parameter"

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?