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

Fiche metier ScolariteV2 #1741

Merged
merged 1 commit into from
Dec 17, 2024
Merged

Fiche metier ScolariteV2 #1741

merged 1 commit into from
Dec 17, 2024

Conversation

Samuelfaure
Copy link
Contributor

No description provided.

@Samuelfaure Samuelfaure requested a review from DorineLam December 4, 2024 13:12
@DorineLam
Copy link
Contributor

@Samuelfaure est-ce qu'on ne peut pas mettre ce yaml dans le même yaml que la version précédente ? Je crois que c'est ce qu'on a fait ailleurs ?

@Samuelfaure Samuelfaure force-pushed the new_scolarite/fiche_metier branch from eb5b34c to 8de579b Compare December 17, 2024 09:09
@DorineLam
Copy link
Contributor

@Miryad3108 est-ce que le contenu de cette fiche correspond à l'API scolarité V2 ? Si oui, on peut merger.

@Miryad3108
Copy link
Contributor

Miryad3108 commented Dec 17, 2024

ça me va oui, merci. Juste je vois ligne 142
call_id:
- Identité pivot
- FranceConnect

alors que l'API n'est pas FranceConnectée, c'est normal @Samuelfaure ?
J'ai vu que c'était aussi le cas dans l'API de la v1 quand je suis partie de ça pour faire la doc de la v2 mais je n'y ai pas touché

@DorineLam
Copy link
Contributor

ça me va oui, merci. Juste je vois ligne 142 call_id: - Identité pivot - FranceConnect

alors que l'API n'est pas FranceConnectée, c'est normal @Samuelfaure ? J'ai vu que c'était aussi le cas dans l'API de la v1 quand je suis partie de ça pour faire la doc de la v2 mais je n'y ai pas touché

C'est une erreur de ma part alors

@Samuelfaure Samuelfaure force-pushed the new_scolarite/fiche_metier branch from 8de579b to bb27d96 Compare December 17, 2024 09:30
@Samuelfaure
Copy link
Contributor Author

Reste plus qu'à relancer les tests quand le swagger sera à jour

@Samuelfaure Samuelfaure force-pushed the new_scolarite/fiche_metier branch from bb27d96 to d1c28bb Compare December 17, 2024 09:32
@Samuelfaure Samuelfaure changed the title Template fiche metier ScolariteV2 Fiche metier ScolariteV2 Dec 17, 2024
@Samuelfaure Samuelfaure force-pushed the new_scolarite/fiche_metier branch from d1c28bb to 9d4974c Compare December 17, 2024 09:41
@Samuelfaure Samuelfaure merged commit 38a66c2 into develop Dec 17, 2024
9 checks passed
@Samuelfaure Samuelfaure deleted the new_scolarite/fiche_metier branch December 17, 2024 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants