-
Notifications
You must be signed in to change notification settings - Fork 33
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
docs(multikeys): met à jour README pour usage de clés multiples #318
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quelques petites suggestions, sinon OK pour la mise à jour de la doc 👍
doc/README-leaflet.md
Outdated
@@ -207,7 +213,7 @@ Votre utilisation des fonctionnalités de l'extension Géoportail sera alors sim | |||
<script> | |||
window.onload = function () { | |||
Gp.Services.getConfig({ | |||
apiKey: 'CLEAPI', | |||
apiKey: 'VOTRE-CLEF', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tu peux rajouter la possibilité de passer plusieurs clés dans le GetConfig ? (et indiquer "CLEF" (ou key) plutôt que "VOTRE-CLEF")
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A faire une fois release extension leaflet réalisée : la dernière release ne prend pas en compte les modifications multikeys
Pull request checklist
Verifiez que votre Pull Request remplit les conditions suivantes :
npm run build
) a été lancé localement et s'est correctement déroulénpm run samples
) ont été testés et validés localementnpm run test
) sont passés localementType de Pull request
Quel type de changement cette Pull Request introduit-elle :
Quel est le comportement actuel (avant PR) :
Numéro du ticket : N/A
Quel est le nouveau comportement :
Documentation des README pour le chargement de nos interfaces en multikeys.
Cette PR introduit-elle des breaking changes ?
Autres informations
Les comportements sont différents selon les APIs sous jacentes :