Skip to content

Conversation

@lionelB
Copy link
Member

@lionelB lionelB commented Jun 23, 2025

🔆 Problème

Actuellement, l'api maddo pour récupérer les participations renvoie toutes les participations (de manière paginée) mais cela reste pénible pour les partenaires pour faire de la synchro.

⛱️ Proposition

On souhaite que l'api maddo puisse remonter seulement les dernières participations mises à jour depuis une certaine date.

🌊 Remarques

On introduit un paramètre since qui prend un timestamp JS (nombre de milliseconde depuis 1 janv 1970)

On vu qu'il manquait des tests d'integ sur les couches repo / usecase / controller coté maddo (l'api reste couverte avec des tests en acceptance)

🏄 Pour tester

ACCESS_TOKEN=$(curl -X 'POST' \
  'https://pix-api-maddo-review-pr12625.osc-fr1.scalingo.io/api/application/token' \
  -s -H 'accept: application/json' \
  -H 'Content-Type: application/x-www-form-urlencoded' \
  -d 'grant_type=client_credentials&client_id=maddo-client&client_secret=maddo-secret&scope=meta campaigns' | jq -r .access_token)

Appeler la route /api/campaigns/:id/participations avec le token et en remplaçant l'id de campagne par celui d'une campagne ayant des participations.

Sans since

Campagne de collecte de profils :

curl --get https://pix-api-maddo-review-pr12625.osc-fr1.scalingo.io/api/campaigns/105141/participations -H "Authorization: Bearer $ACCESS_TOKEN" | jq .

Campagne d'évaluation :

curl https://pix-api-maddo-review-pr12625.osc-fr1.scalingo.io/api/campaigns/104906/participations -H "Authorization: Bearer $ACCESS_TOKEN" | jq .

Avec since

Campagne de collecte de profils :

curl --get --data-urlencode "since=2025-06-24" https://pix-api-maddo-review-pr12625.osc-fr1.scalingo.io/api/campaigns/105141/participations -H "Authorization: Bearer $ACCESS_TOKEN" | jq .

Campagne d'évaluation :

 curl --get --data-urlencode "since=2025-06-24" https://pix-api-maddo-review-pr12625.osc-fr1.scalingo.io/api/campaigns/104906/participations -H "Authorization: Bearer $ACCESS_TOKEN" | jq .

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@VincentHardouin VincentHardouin changed the title [FEATURE] ajoute un filtre sur l'api maddo pour remonter les dernières participations mise à jour (PIX-18288) [FEATURE] Ajouter un filtre sur l'api maddo pour remonter les dernières participations mise à jour (PIX-18288) Jun 24, 2025
@lionelB lionelB force-pushed the PIX-18288/add-cursor-on-campaign-participations-api branch 3 times, most recently from 5e70277 to 518ffa6 Compare June 24, 2025 09:13
@lionelB lionelB added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed Need PO validation for this functionally labels Jun 24, 2025
@lionelB lionelB force-pushed the PIX-18288/add-cursor-on-campaign-participations-api branch from 518ffa6 to 53a986a Compare June 24, 2025 09:32
@pix-service-auto-merge pix-service-auto-merge force-pushed the PIX-18288/add-cursor-on-campaign-participations-api branch from 53a986a to d6a8437 Compare June 24, 2025 12:01
@pix-service-auto-merge pix-service-auto-merge merged commit e72d72d into dev Jun 24, 2025
10 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the PIX-18288/add-cursor-on-campaign-participations-api branch June 24, 2025 12:06
pix-service pushed a commit that referenced this pull request Jun 25, 2025
# [5.143.0](v5.142.0...v5.143.0) (2025-06-25)

### 🚀 Amélioration

- [#12604](#12604) Afficher une modale avant de quitter la page de fin de parcours (PIX-18023)
- [#12625](#12625) Ajouter un filtre sur l'api maddo pour remonter les dernières participations mise à jour (PIX-18288)
- [#12626](#12626) ajouter une route PATCH users/{id} pour enrichir un utilisateur anonyme avec des données d'authentification (Pix-18022)
- [#12639](#12639) Détacher les profile-rewards de l'organisation lors de la supression de learner (PIX-18113).
- [#12640](#12640) Lister par défaut les feature-toggles.
- [#12619](#12619) Repenser l'affichage des niveaux sur la page d'analyse de Pix Orga (PIX-18337)
- [#12635](#12635) v2 du module tmp-ia-fonctionnement-debut pour revue PYO

### 🐛 Correction

- [#12632](#12632) Corriger une typo dans le message de MER
- [#12528](#12528) Mettre à jour les informations d'une invitation à rejoindre une organisation lors d'un renvoie avec de nouveaux paramètres (PIX-18065)
- [#12621](#12621) Réparer l'affichage du nombre de tentatives de connexions restantes (PIX-18405)

### 🏗️ Tech

- [#12627](#12627) Bump audit-logger dependencies
- [#12568](#12568) Montée de version EmberJS pour PixAdmin

### ☕ Autre

- [#12638](#12638) Corriger l'exemple de titre de PR
@pix-service
Copy link
Contributor

🎉 This PR is included in version 5.143.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants