Skip to content

🐛(web-ingestion) fix upsert offers payload in API#693

Open
AntoineAugusti wants to merge 1 commit into
mainfrom
fix/offers-upsert-view-payload
Open

🐛(web-ingestion) fix upsert offers payload in API#693
AntoineAugusti wants to merge 1 commit into
mainfrom
fix/offers-upsert-view-payload

Conversation

@AntoineAugusti
Copy link
Copy Markdown
Member

📝 Description

Contexte

L'implémentation de OffersUpsertView était incohérente avec sa propre documentation OpenAPI (voir #691). La vue attendait un tableau brut
[offre1, offre2] comme corps de requête, alors que le schéma documente {"offres": [offre1, offre2]}.

Changements

OffersUpsertView

  • Le serializer de validation reçoit désormais request.data directement (plus de double-enveloppe manuelle)
  • L'itération sur les offres utilise request.data["offres"] au lieu de request.data

Tests

  • Tous les appels data=[...] mis à jour en data={"offres": [...]} pour correspondre au format documenté

🏷️ Type de changement

  • 🐛 Correction de bug
  • 🎢 Nouvelle fonctionnalité (changement non bloquant qui ajoute une fonctionnalité)
  • 🥁 Changement breaking (modification ou fonctionnalité qui pourrait casser le fonctionnement existant) nécessitant une mise à jour de la documentation
  • 📚 Mise à jour de la documentation
  • ♻️ Refactorisation
  • 🔧 Changement technique

🔧 Modifications

Lister les changements principaux

🛸 Dépendances requises pour ce changement (si applicable)

🏝️ Comment tester (si applicable)

Étapes pour reproduire ou tester

📸 Captures d’écran (si applicable)

✅ Liste de contrôle

  • 💅 J’ai ajouté ou mis à jour les tests appropriés.
  • 📝 J’ai mis à jour ou ajouté la documentation nécessaire.
  • 🚀 J’ai pris en compte l’impact sur les performances, la sécurité et l’expérience utilisateur.
  • 👀 J’ai demandé une revue à une personne de l’équipe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix PR: scoped bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: OffersUpsertView implementation inconsistent with OpenAPI spec

1 participant