You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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 :
Feature
Quel est le comportement actuel (avant PR) :
Refonte et modification du composant Drawing (nouveau composant Draw).
Numéro du ticket : N/A (à définir)
Quel est le nouveau comportement :
Refonte du composant Draw en partant d'un composant côté ext-gpf "bas niveau" Toggle (bouton simple avec changement d'état), ToggleInteraction (bouton contenant une interaction openlayers) et ToggleContent (bouton contrôlant un panneau / dialog)
Cette PR introduit-elle des breaking changes ?
Oui
Liste des breakings change à lister ici
Retrait de la méthode setCollapsed et de l'événement change:collapsed : à la place, un setActive et un événement change:active ont été ajoutées (+ en accord avec les méthodes openlayers).
Warning
Outre le changement de nom, c'est aussi un changement de comportement : il faut remplacer setCollapsed(true) (ferme la modale) par setActive(false) (désactive le contrôle).
@elias75015 dans le contrôle Draw (le "nouveau"), il n'y a pas de question de formats d'exports et je pensais que ça ferait sens vu que vous utilisez un contrôle d'export pour le bouton exporter et enregistrer.
Mais dans la fonction onToggleShowVector de Drawing.vue, pour enregistrer temporairement un dessin si l'user n'est pas connecté, les méthodes exportFeatures, getExportName et getExportFormat sont utilisés. Ces méthodes ne sont pour le moment pas implémentées dans le contrôle Draw. En soit les implémenter est possible, il suffit de reprendre la même logique, mais il ne serait pas possible plutôt de passer par le contrôle d'Export plutôt ? Et d'utiliser la même logique que pour un export du dessin ?
Si pas possible, lourd ou pas adapté, je modifierais le composant Draw.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull request checklist
À faire / tester :
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) :
Refonte et modification du composant Drawing (nouveau composant Draw).
Numéro du ticket : N/A (à définir)
Quel est le nouveau comportement :
Toggle(bouton simple avec changement d'état),ToggleInteraction(bouton contenant une interaction openlayers) etToggleContent(bouton contrôlant un panneau / dialog)Cette PR introduit-elle des breaking changes ?
Liste des breakings change à lister ici
setCollapsedet de l'événementchange:collapsed: à la place, unsetActiveet un événementchange:activeont été ajoutées (+ en accord avec les méthodes openlayers).Warning
Outre le changement de nom, c'est aussi un changement de comportement : il faut remplacer
setCollapsed(true)(ferme la modale) parsetActive(false)(désactive le contrôle).Autres informations
Lié à la PR IGNF/geopf-extensions-openlayers#490
Choses à vérifier / tests à effectuer