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

Mise à jour de la documentation pour l'ajout de couches mapbox en 2D et 3D #55

Merged
merged 2 commits into from
Apr 15, 2020

Conversation

elias75015
Copy link
Contributor

@elias75015 elias75015 commented Apr 15, 2020

Met à jour la documentation sur les paramètres à passer en options pour ajouter une couche mapBox en 2D ou en 3D

Pull request checklist

Verifiez que votre Pull Request remplit les conditions suivantes :

  • Des tests ont été ajoutés pour les changements (corrections de bugs ou features)
  • De la documentation a été mise à jour ou ajoutée si nécessaire (corrections de bugs ou features)
  • Un build (npm run build) a été lancé localement et s'est correctement déroulé
  • Les exemples impactés par les modifications (npm run samples) ont été testés et validés localement
  • Les tests (npm run test) sont passés localement

Type de Pull request

Quel type de changement cette Pull Request introduit-elle :

  • Bugfix
  • Feature
  • Mise à jour du style du code (syntaxe, renommage de fonctions)
  • Refactoring (lisibilité/performance du code, sans changements fonctionnels)
  • Changement sur le processus de build
  • Contenu de la documentation
  • Autres (décrire ci-après) :

Quel est le comportement actuel (avant PR) :

Documentation incomplète sur l'ajout de couches mapbox.

Quel est le nouveau comportement :

Documentation mise à jour, notamment sur les paramètres en entrée pour l'ajout de couches mapbox.

Cette PR introduit-elle des breaking changes ?

  • Oui
  • Non

Autres informations

cf. ticket RDEV-34145

Mets à jour les paramètres à passer en options pour ajouter une couche mapBox en 2D ou en 3D

cf ticket RDEV-34145
@elias75015
Copy link
Contributor Author

elias75015 commented Apr 15, 2020

Quelques questions, pour compléter cette PR. Les paramètres suivants sont-ils utilisés pour les couches mapBox, et si oui, à quoi servent-ils ?

  • outputFormat
  • projection
  • stylesSummary
  • filtersSummary

@lowzonenose
Copy link
Contributor

  • outputFormat

non utilisé pour la 2D

  • projection

non utilisé pour la 2D

  • stylesSummary

descriptif des thèmes.
via les options de l'éditeur de styles, on a la possibilité de l'afficher...
non utilisé dans le portail.

  • filtersSummary

descriptif des thèmes
via les options de l'éditeur de styles, on a la possibilité de l'afficher...
non utilisé dans le portail.

Copy link
Contributor

@lowzonenose lowzonenose left a comment

Choose a reason for hiding this comment

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

l'option urlService devrait être utilisée dans un futur proche afin de pouvoir surcharger l'url du service de vecteur tuilé...

@elias75015
Copy link
Contributor Author

OK, je retire les options non utilisées, ajoute la description des "Summary", et réintègre urlService en le décrivant comme "à venir"

@elias75015 elias75015 merged commit 4cf552f into develop Apr 15, 2020
@elias75015 elias75015 deleted the feature-mapbox-Jsdoc branch April 15, 2020 17:02
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