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

Erreur pour "Ouvrir le lien externe" lorsque c'est une municipalité #409

Closed
mikesmichael opened this issue Feb 11, 2020 · 14 comments
Closed
Assignees
Labels
bug Incorrect or unexpected behaviors // Anomalies de fonctionnement

Comments

@mikesmichael
Copy link
Contributor

mikesmichael commented Feb 11, 2020

Lorsque l'on recherche une municipalité, on obtient une erreur lorsque l'on clique sur "Ouvrir le lien externe" de GoogleMaps.

Test fait avec: les municipalités:
"Gouvernement régional d'Eeyou Istchee Baie-James" => about:blank#blocked
et
"Maskinongé" => 413. That’s an error. Your client issued a request that was too large. That’s all we know.

@mikesmichael mikesmichael added the bug Incorrect or unexpected behaviors // Anomalies de fonctionnement label Feb 11, 2020
@mbarbeau mbarbeau added this to To do in IGO2 - Roadmap V1.4 via automation May 1, 2020
@PhilippeLafreniere18
Copy link
Collaborator

PhilippeLafreniere18 commented Jun 30, 2020

Même si le reste des entités ne donnent pas nécessairement d'erreurs, presque tous les lignes et polygones n'aboutissent à rien dans google maps en raison de la géométrie complexe (pleins de points lon/lat). Définitivement à travailler

@PhilippeLafreniere18 PhilippeLafreniere18 moved this from To do to In progress in IGO2 - Roadmap V1.4 Jun 30, 2020
@PhilippeLafreniere18 PhilippeLafreniere18 self-assigned this Jun 30, 2020
@PhilippeLafreniere18 PhilippeLafreniere18 moved this from In progress to Review in progress in IGO2 - Roadmap V1.4 Jun 30, 2020
@mbarbeau
Copy link
Member

mbarbeau commented Jun 30, 2020

On a 2 choix possibles :

  1. Passer le centroide de la géométrie = Précision assurée par contre seulement une pin au milieu et n'est pas associé à un élément dans google.
  2. Passer le nom de l'entité = Est associé à un élément dans google mais peut, dans certains cas, nous apporter à un mauvais endroit (noms identiques)

Vote !
Si vous préférez 1, mettez un coeur en réaction ❤️
Si vous préférez 2, mettez une fusée en réaction 🚀

@ameliebernier
Copy link

ameliebernier commented Jun 30, 2020

Pour moi ça serait l'option 2, le but de l'ouvrir dans Google c'est quand même qu'il soit lié à l'élément de google... comme si tu avais fait la recherche dans Google

@mbarbeau
Copy link
Member

mbarbeau commented Jun 30, 2020

Complément d'informations:
Pour les points, on passe les coordonnées. Donc faudrait être cohérent et passer les points avec des noms au lieu de la coordonnée si l'option 2 est choisie.

@mbarbeau
Copy link
Member

mbarbeau commented Jun 30, 2020

Comme sous-question:
Le but de l'ouvrir dans Google est d'avoir l'élément de Google lié ou de l'ouvrir à la bonne place ?

@PhilippeLafreniere18
Copy link
Collaborator

PhilippeLafreniere18 commented Jun 30, 2020

Comme sous-question:
Le but de l'ouvrir dans Google est d'avoir l'élément de Google lié ou de l'ouvrir à la bonne place ?

Je pense que d'ouvrir Google en plein milieu d'une rue qui se trouve au centroïde d'une municipalité par exemple, ne dit pas grand chose... De plus, Google fait aussi une association avec l'adresse la plus proche quand tu lui passes une coordonnée

@mikesmichael
Copy link
Contributor Author

mikesmichael commented Jun 30, 2020

Je pense que je dirais choix 1 aussi. Étant donné que tu recherches à te localiser dans google map je pense que c'est ce qui est le plus important c'est de localiser la position plutôt que l'information. Une fois que tu es localisé, tu pourras poursuivre tes recherches pour obtenir les informations google si tu le désires. Si tu prends l'option 2, tu ne seras pas capable de te retrouver pour poursuivre les recherches. Par exemple si tu cherches une borne sumi dans igo et tu veux la retrouver dans google, l'option 2 va te faire chier car tu ne les retrouveras jamais à partir de ses informations.

@ameliebernier
Copy link

ameliebernier commented Jun 30, 2020

Bon je suis d,accord avec les deux commentaires précédents, en fait le meilleur comportement dépend du type d'élément... pour une mun effectivement, ça ne fait pas de sens d'être localisé à une adresse au centre de celle-ci. Pour une borne par contre, oui on voudrait être sur le point

@mbarbeau
Copy link
Member

mbarbeau commented Jun 30, 2020

Ça complique les affaires si ce n'est pas général:
si borne sumi -> coordonnées
si municipalité -> utiliser designation + nom + regadmin
si route -> utiliser nom + mun
si adresse -> utiliser nom + mun + code postale
si lieu -> ?
si ... -> ....

@mikesmichael
Copy link
Contributor Author

mikesmichael commented Jun 30, 2020

Faire les 2 options dans le menu, genre:
recherche maps avec localisation
recherche maps avec détails

Bonne idée?

@ameliebernier
Copy link

ameliebernier commented Jun 30, 2020

Pour l'adresse, je ne pense pas que ça serait un problème d'utiliser la coordonnée...

Mais sinon on retire le lien Google Maps quand ça fait pas de sens? Car pour une municipalité, ça ne fait pas de sens de localiser un point, ce n'est pas utile.

@PhilippeLafreniere18
Copy link
Collaborator

PhilippeLafreniere18 commented Jun 30, 2020

On peut garder la coordonnée pour les points et faire un custom pour les lignes et les polygones avec nom ou simplement l'enlever pour les lignes et polygones... Si on veut garder ça générique

@mikesmichael
Copy link
Contributor Author

mikesmichael commented Jun 30, 2020

Je pense que de laisser le choix à l'utilisateur de chercher soit la description ou la localisation ça reste le plus simple (à faire, à utiliser, à configurer) et ça laisse la flexibilité de pouvoir faire les 2 recherches selon ce que l'utilisateur à envie de faire. Si on commence à paramétrer pour chacun des types de recherche c'est sur que ça va arriver qu'un utilisateur va vouloir la localisation pour une recherche X et qu'un autre voudra la description.

@PhilippeLafreniere18 PhilippeLafreniere18 moved this from Review in progress to Reviewer approved in IGO2 - Roadmap V1.4 Jul 3, 2020
@mbarbeau
Copy link
Member

mbarbeau commented Jul 8, 2020

@mbarbeau mbarbeau closed this as completed Jul 8, 2020
IGO2 - Roadmap V1.4 automation moved this from Reviewer approved to Done Jul 8, 2020
matrottier pushed a commit that referenced this issue Nov 17, 2021
* ui(miniMap): update miniMap title when more than 2 baselayers

* feat(title): move logic to baselayers-switcher
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Incorrect or unexpected behaviors // Anomalies de fonctionnement
Projects
No open projects
Development

No branches or pull requests

4 participants