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

Ajout APIdev Millibase #1303

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Ajout APIdev Millibase #1303

wants to merge 3 commits into from

Conversation

Miryad3108
Copy link
Contributor

  • Ajout de contenu

@Miryad3108 Miryad3108 marked this pull request as ready for review January 11, 2024 13:13
@@ -29,6 +29,10 @@ export const pathEligible = {
answer: `<br/><Button href="https://datapass.api.gouv.fr/api-particulier?demarche=agora-plus">Remplir une demande</Button>
`,
},
'APIdev (Millibase)',
],
answer: `<br/><<Button href="https://datapass.api.gouv.fr/api-particulier?demarche=APIdev-Millibase">Remplir une demande</Button>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ce lien renvoie vers une demande libre ? Est-ce normal ?
Retirer les majuscule dans l'URL ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En fait je n'arriva pas à comprendre où est-ce que tu as configuré l'URL.
Le morceau d'URL apidev-millibase doit figurer dans ton code, comme une clé. Or là il n'y a rien dans cette PR. Il faudrait déjà que tu regroupes cette PR et celle-ci : https://github.com/betagouv/datapass/pull/1369/files
Car elles vont ensemble et autrement c'est difficile de comprendre.

De plus, je pense que ta clé Apidev dans la PR #1369 est justement cette clé qui doit figurer dans l'URL. À mon sens, tu devrais remplacer Apidev par apidev-millibase et utiliser l'URL suivante :
https://datapass.api.gouv.fr/api-particulier?demarche=APIdev-Millibase

À revérifier car j'ai regardé rapidement, pour t'aider tu peux chercher à faire comme pour un autre formulaire, par exemple ccas-arpege

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.

2 participants