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

toulouse-maelis: add service criteria to catalog (#77084) #245

Merged
nroche merged 1 commits from wip/77084-parsifal-add-service-criteria into main 2023-05-31 10:47:13 +02:00
Conversation 2 Commits 1 Files Changed 3 +337 -11
nroche commented 2023-05-10 19:05:53 +02:00
Owner
Copy Link

Avoir un nouveau critère "service" sur le catalogue général.

Je propose ce patch à la relecture bien que je devrais le découper pour expliciter la prise en charge des critères vides et aussi compléter les tests avec de nouvelles activités
(J'ai du mal à avoir des données de tests).

Avoir un nouveau critère "service" sur le catalogue général. Je propose ce patch à la relecture bien que je devrais le découper pour expliciter la prise en charge des critères vides et aussi compléter les tests avec de nouvelles activités (J'ai du mal à avoir des données de tests).
fpeters commented 2023-05-13 15:52:57 +02:00
Owner
Copy Link

Je propose ce patch à la relecture bien que je devrais le découper pour expliciter la prise en charge des critères vides et aussi compléter les tests avec de nouvelles activités

Je pense qu'il vaut mieux attendre que ça soit vraiment prêt à être relu.

(mais toujours, s'il y a un endroit particulier où tu veux un retour, tu peux le pointer).

> Je propose ce patch à la relecture bien que je devrais le découper pour expliciter la prise en charge des critères vides et aussi compléter les tests avec de nouvelles activités Je pense qu'il vaut mieux attendre que ça soit vraiment prêt à être relu. (mais toujours, s'il y a un endroit particulier où tu veux un retour, tu peux le pointer).
nroche force-pushed wip/77084-parsifal-add-service-criteria from dbd659cbeb to 54715a3cac 2023-05-16 16:52:41 +02:00 Compare
nroche commented 2023-05-16 17:06:54 +02:00
Author
Owner
Copy Link

Prêt à la relecture : j'ai complété les tests avec de nouvelles activités.

(j'ai finalement choisi de ne pas découper en plusieurs patchs pour expliciter la prise en charge des critères vides, parce que les patchs se re-couvriraient d'une part et aussi parce que j'ai l'impression que la relecture par patch n'est plus trop d'usage avec le passage à gitea)

Prêt à la relecture : j'ai complété les tests avec de nouvelles activités. (j'ai finalement choisi de ne pas découper en plusieurs patchs pour expliciter la prise en charge des critères vides, parce que les patchs se re-couvriraient d'une part et aussi parce que j'ai l'impression que la relecture par patch n'est plus trop d'usage avec le passage à gitea)
fpeters approved these changes 2023-05-30 16:15:16 +02:00
nroche force-pushed wip/77084-parsifal-add-service-criteria from 54715a3cac to d176d9fc4b 2023-05-31 10:36:00 +02:00 Compare
nroche merged commit d176d9fc4b into main 2023-05-31 10:47:13 +02:00
nroche deleted branch wip/77084-parsifal-add-service-criteria 2023-05-31 10:47:13 +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/passerelle#245
No description provided.
Delete Branch "wip/77084-parsifal-add-service-criteria"

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?