Skip to content

[FEATURE] Création de la route pour récupérer les prescrits dans Pix Admin (PIX-23150)#16614

Merged
SimonBertaux merged 8 commits into
devfrom
pix-23150-create-admin-learners-list-route
Jun 26, 2026
Merged

[FEATURE] Création de la route pour récupérer les prescrits dans Pix Admin (PIX-23150)#16614
SimonBertaux merged 8 commits into
devfrom
pix-23150-create-admin-learners-list-route

Conversation

@SimonBertaux

@SimonBertaux SimonBertaux commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

🪧 Problème

Le support ne peut pas rechercher des prescrits sans utiliser metabase. Il n’existe pas encore de routes de récupération des prescrits pour Pix Admin.

🌈 Proposition

Créer la route dans l’API. Investiguer pour utiliser le modèle OrganizationLearners et le repository déjà existants et voir s’ils ont besoin d'être modifiés.

La route ne devra pas retourner de prescrits deleted (mais les désactivés sont retournés).

✊ Remarques

Aucune limite de récupération n'a été fixée dans la requête car il existe apparemment un comportement défini pour ça.

🎉 Pour tester

Tests au vert.

Faire un curl vers la route pour vérifier qu’elle renvoie les bonnes infos.

ACCESS_TOKEN=$(curl -X 'POST' -s  'https://api-pr16614.review.pix.fr/api/token' -d 'grant_type=password&username=superadmin@example.net&password=pix123' | jq -r ".access_token")

curl -H "Authorization: Bearer $ACCESS_TOKEN" "https://api-pr16614.review.pix.fr/api/admin/organization-learners?page%5Bnumber%5D=1&page%5Bsize%5D=50&filter%5BfullName%5D=Zoro&filter%5BorganizationId%5D=9001" | jq

@SimonBertaux SimonBertaux requested a review from a team as a code owner June 23, 2026 15:50
@SimonBertaux SimonBertaux added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally team-prescription Adopt US - La Team qui mange et parle fort labels Jun 23, 2026
@pix-bot-github

pix-bot-github commented Jun 23, 2026

Copy link
Copy Markdown

@SimonBertaux SimonBertaux force-pushed the pix-23150-create-admin-learners-list-route branch from 6739ba2 to e66e9d9 Compare June 23, 2026 15:54
@baptiste-h baptiste-h force-pushed the pix-23150-create-admin-learners-list-route branch 2 times, most recently from acf607c to 60147d0 Compare June 24, 2026 16:42
Comment thread api/src/prescription/organization-learner/application/learner-list-route.js Outdated
@SimonBertaux SimonBertaux force-pushed the pix-23150-create-admin-learners-list-route branch from 60147d0 to b61c9a0 Compare June 26, 2026 12:49
@SimonBertaux SimonBertaux added Func Review OK PO validated functionally the PR Tech Review OK and removed 👀 Func Review Needed Need PO validation for this functionally 👀 Tech Review Needed labels Jun 26, 2026
@SimonBertaux SimonBertaux force-pushed the pix-23150-create-admin-learners-list-route branch from b61c9a0 to 80b2ae5 Compare June 26, 2026 12:59
@SimonBertaux SimonBertaux enabled auto-merge June 26, 2026 13:01
@SimonBertaux SimonBertaux added this pull request to the merge queue Jun 26, 2026
Merged via the queue into dev with commit 5466d09 Jun 26, 2026
9 checks passed
@SimonBertaux SimonBertaux deleted the pix-23150-create-admin-learners-list-route branch June 26, 2026 13:14
pix-service pushed a commit that referenced this pull request Jun 26, 2026
# [5.442.0](v5.441.0...v5.442.0) (2026-06-26)

### 🚀 Amélioration

- [#16642](#16642) Ajouter une route pour vérifier si un utilisateur a répondu au questionnaire NPS d'une campagne (PIX-23304)
- [#16614](#16614) Création de la route pour récupérer les prescrits dans Pix Admin (PIX-23150)
- [#16658](#16658) Partager les attestations des parcours combinés avec les organisations (PIX-21895)

### 🐛 Correction

- [#16651](#16651) Répare un bug sur les labels d'attestations dans PixApp (PIX-23353)

### 🏗️ Tech

- [#16654](#16654) Déplacer les tests unitaires dans leurs contextes respectifs
@pix-service

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 5.442.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

SimonBertaux added a commit that referenced this pull request Jun 26, 2026
…Admin (PIX-23150) (#16614)

## 🪧 Problème

Le support ne peut pas rechercher des prescrits sans utiliser metabase.
Il n’existe pas encore de routes de récupération des prescrits pour Pix
Admin.

## 🌈 Proposition

Créer la route dans l’API. Investiguer pour utiliser le modèle
OrganizationLearners et le repository déjà existants et voir s’ils ont
besoin d'être modifiés.

La route ne devra pas retourner de prescrits deleted (mais les
désactivés sont retournés).

## ✊ Remarques

Aucune limite de récupération n'a été fixée dans la requête car il
existe apparemment un comportement défini pour ça.

## 🎉 Pour tester
Tests au vert.

Faire un curl vers la route pour vérifier qu’elle renvoie les bonnes
infos.

```
ACCESS_TOKEN=$(curl -X 'POST' -s  'https://api-pr16614.review.pix.fr/api/token' -d 'grant_type=password&username=superadmin@example.net&password=pix123' | jq -r ".access_token")

curl -H "Authorization: Bearer $ACCESS_TOKEN" "https://api-pr16614.review.pix.fr/api/admin/organization-learners?page%5Bnumber%5D=1&page%5Bsize%5D=50&filter%5BfullName%5D=Zoro&filter%5BorganizationId%5D=9001" | jq
```

---------

Co-authored-by: Baptiste H <baptiste.hetet@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Func Review OK PO validated functionally the PR released team-prescription Adopt US - La Team qui mange et parle fort Tech Review OK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants