map: init map position from lat;lon in template (#87633) #1207

nroche · 2024-02-29T16:19:19+01:00

nroche commented

2024-02-29 16:19:19 +01:00

#66959 me semble avoir été qualifié uniquement avec les adresses :
https://demarches.eservices.toulouse-metropole.fr/backoffice/forms/16/fields/pages/2b9e37af-61fb-46e1-bf0a-f559ee8d74dd/6b7fa3d5-c69d-437c-9336-292759c20606/
Ce patch permet aussi de passer les valeurs en dur.
Or pour les adresses en dûr on ne passe pas par ce chemin là :

           } else if ($map_widget.data('def-template')) {
             // do not adjust map to fit markers as a specific location string
             // has been given.

#66959 me semble avoir été qualifié uniquement avec les adresses : https://demarches.eservices.toulouse-metropole.fr/backoffice/forms/16/fields/pages/2b9e37af-61fb-46e1-bf0a-f559ee8d74dd/6b7fa3d5-c69d-437c-9336-292759c20606/ Ce patch permet aussi de passer les valeurs en dur. Or pour les adresses en dûr on ne passe pas par ce chemin là : ``` } else if ($map_widget.data('def-template')) { // do not adjust map to fit markers as a specific location string // has been given. ```

nroche commented

2024-02-29 16:20:05 +01:00

Je n'ai pas réussi à écrire le test. Je veux bien passer la main s'il est requis.

fpeters requested changes 2024-02-29 16:35:30 +01:00

fpeters left a comment

#1196 fait ça aussi, et contient un test, il faudrait se coordonner, peut-être limiter l'autre PR à juste la partie JS, encore discutée.

Aussi, il faudrait revoir le sujet du commit, plutôt de l'ordre de "misc: do not adjust map to fit markers when a specific center is set (#.....)"

https://git.entrouvert.org/entrouvert/wcs/pulls/1196 fait ça aussi, et contient un test, il faudrait se coordonner, peut-être limiter l'autre PR à juste la partie JS, encore discutée. Aussi, il faudrait revoir le sujet du commit, plutôt de l'ordre de "misc: do not adjust map to fit markers when a specific center is set (#.....)"

nroche force-pushed wip/87633-init-map-position-from-template from 7dbe29c25a to 1d7e503fb1

2024-02-29 17:43:32 +01:00

Compare

nroche commented

2024-02-29 17:46:28 +01:00

#1196 fait ça aussi,
peut-être limiter l'autre PR à juste la partie JS, encore discutée.

Zut, j'ai raté ça.
Je vous laisse la main (et la PR) si vous voulez.

et contient un test, il faudrait se coordonner,

Merci, je l'ai intégré.

Aussi, il faudrait revoir le sujet du commit, plutôt de l'ordre de "misc: do not adjust map to fit markers when a specific center is set (#.....)"

Fait.

> https://git.entrouvert.org/entrouvert/wcs/pulls/1196 fait ça aussi, > peut-être limiter l'autre PR à juste la partie JS, encore discutée. Zut, j'ai raté ça. Je vous laisse la main (et la PR) si vous voulez. > et contient un test, il faudrait se coordonner, Merci, je l'ai intégré. > Aussi, il faudrait revoir le sujet du commit, plutôt de l'ordre de "misc: do not adjust map to fit markers when a specific center is set (#.....)" Fait.

csechet approved these changes 2024-02-29 17:56:59 +01:00

csechet commented

2024-02-29 17:57:26 +01:00

Vas-y Nico, parce que je sais pas encore ce qu'on va faire de ma PR, je rebaserai.

nroche merged commit eaf83221fb into main

2024-02-29 18:04:41 +01:00

nroche deleted branch wip/87633-init-map-position-from-template

2024-02-29 18:04:41 +01:00

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No Assignees

3 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#1207