Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE] Connecte un datamart externe pour servir les résultats de certification pour ParcourSup (PIX-15800). #10896

Merged

Conversation

HEYGUL
Copy link
Contributor

@HEYGUL HEYGUL commented Dec 24, 2024

🎄 Problème

Les données de résultat de certification étant des données "froides" remplies dans un datamart externe à l'api Pix, il est nécessaire de pouvoir s'y connecter.

🎁 Proposition

On ajouter la notion de datamart en parallèle à la notion de base de données.
Via la variable d'environnement DATAMART_DATABASE_URL il est alors possible de servir des données depuis une base de données externe.

🧦 Remarques

Beaucoup de code dupliqué pour accéder, créer et remplir le DATAMART, à voir comment on veut mutualiser.

🎅 Pour tester

Après avoir valoriser la DATAMART_DATABASE_URL avec le datamart d'intégration.

Sur https://api-pr10896.review.pix.fr/api/documentation, générer un token valide avec les bons client_id, client_secret et scope

TOKEN=<token>
curl https://api-pr10896.review.pix.fr/api/parcoursup/students/999659255FT/certification \
    -H "Authorization: Bearer $TOKEN" \
    -H "Content-Type: application/json" \
    -X GET

doit retourner

{ "ine": "999659255FT" }

Tester également avec un INE valide mais non présent dans le datamart, eg. "123456789OK".
Vérifier qu'une erreur 404 est retournée.

@HEYGUL HEYGUL self-assigned this Dec 24, 2024
@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 :

@HEYGUL HEYGUL force-pushed the pix-15800-get-parcoursup-data-from-datamart branch 4 times, most recently from e5bfbb6 to 5332742 Compare December 24, 2024 11:41
@HEYGUL HEYGUL requested review from a team as code owners December 24, 2024 11:41
@HEYGUL HEYGUL force-pushed the pix-15800-get-parcoursup-data-from-datamart branch 5 times, most recently from cf0b303 to b6a0d95 Compare December 26, 2024 08:09
@HEYGUL HEYGUL force-pushed the pix-15800-get-parcoursup-data-from-datamart branch from b6a0d95 to 2370bc4 Compare December 26, 2024 09:22
@yaf yaf force-pushed the pix-15800-get-parcoursup-data-from-datamart branch from f21c865 to 1d0242a Compare December 26, 2024 10:07
@HEYGUL HEYGUL force-pushed the pix-15800-get-parcoursup-data-from-datamart branch from 1d0242a to b8a5d8c Compare December 26, 2024 10:18
@HEYGUL HEYGUL force-pushed the pix-15800-get-parcoursup-data-from-datamart branch from b8a5d8c to 442c39a Compare December 26, 2024 10:31
HEYGUL and others added 3 commits December 26, 2024 16:18
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-15800-get-parcoursup-data-from-datamart branch from 442c39a to 55b127a Compare December 26, 2024 16:18
@pix-service-auto-merge pix-service-auto-merge merged commit ee6e7e4 into dev Dec 26, 2024
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15800-get-parcoursup-data-from-datamart branch December 26, 2024 16:24
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.

4 participants