Skip to content
This repository has been archived by the owner on Oct 22, 2024. It is now read-only.

API v2 ServiceSerializer: on ne filtre plus les sous-catégories --autre #392

Merged
merged 2 commits into from
Oct 1, 2024

Conversation

ggounot
Copy link
Collaborator

@ggounot ggounot commented Oct 1, 2024

DI fait dorénavant lui-même la transformation. On ne filtre donc plus les sous-catégories se terminant par --autre.

Le test de ce filtrage a été supprimé.

https://trello.com/c/aLeDF7Gj/199-di-x-dora-api-ne-renvoi-pas-les-th%C3%A9matiques-dun-service#comment-66f513ee9701b3ba18cf1193

@ggounot
Copy link
Collaborator Author

ggounot commented Oct 1, 2024

@ggounot ggounot requested review from ikarius and jbuget October 1, 2024 12:54
Copy link
Contributor

@jbuget jbuget left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@ggounot ggounot merged commit ec4653e into main Oct 1, 2024
6 checks passed
@ggounot ggounot deleted the exposition-services-thematiques-autre branch October 1, 2024 13:51
jbuget added a commit that referenced this pull request Oct 9, 2024
jbuget added a commit that referenced this pull request Oct 9, 2024
### 🍣 Contexte / problème

La PR #392 modifie les données envoyées par DORA à DI via l'API. 

Nous nous sommes mal synchronisés / organisés entre les 2 équipes.

Nous avons envoyé la modif alors que DI n'a pas fait la leur de leur
côté.

Ça génère des erreurs et des services qui ne sont pas remontés dans DI.

### 🦄 Solution

Le temps de permettre à DI de faire le fix, nous proposons de rollback
la modif.

### 🍿 Misc

This reverts commit ec4653e.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants