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
feat(bouquet): spatial coverage #393
Conversation
✅ Deploy Preview for ecospheres ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for meteo-france ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
69e1220
to
430d42b
Compare
430d42b
to
f9ba779
Compare
f9ba779
to
fa3e337
Compare
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.
Excellent !
@Thesauruv pour info :)
Quelques problèmes mineurs :
J'ai buggé sur le "11" au début. Pour les départements c'est évident, pour le reste pas tant que ça. Est-ce qu'on indiquerait plutôt "(INSEE: 11)" (avec INSEE en petite font) ?
Possible de limiter aux territoires français ?
Problème de rémanence 1 :
- J'ai déjà sélectionné une géozone (IDF).
- J'édite le bouquet pour la modifier.
- Je clique dans le champ spatial.
- La géozone existante ne disparait pas.
Problème de rémanence 2 :
- Je n'ai pas de géozone.
- Je clique dans le champ spatial et je sélectionne une géozone (Bourgogne).
- Je supprime la géozone via la croix à droite du champ.
- Les suggestions précédentes s'affichent encore en drop down.
Sinon le suggest est pourri (seulement de l'exact match), mais ça il faudra voir avec data.gouv.
On peut aussi se passer du code, voire du niveau, je pense. Après tout je ne l'ai pas repris sur les autres affichages. Je pense qu'il est surtout utile dans la liste de sélection si jamais il y a un doute entre plusieurs valeurs (et on doit pouvoir chercher par code).
Non je ne pense pas.
Vu, en effet. Je vais voir ce que je peux faire, mais ce n'est pas strictement lié à cette PR (le select jdd se comporte pareil). Donc de toute façon une issue subséquente : soit fixer les deux, soit fixer le select jdd si fix ici. |
On peut... Le code en dehors des départements ça me dit rien, mais peut-être que ça aidera des profils plus "administratifs" ? Si on vire code+niveau j'ai peur que ça soit compliqué en cas d'ambiguité.
-> later
Je viens de voir qu'au moins le pb 1 était identique sur le suggest datasets et curieusement ça m'a paru logique à cet endroit là :/ Vu que ça clear le champ dès qu'on commence à taper (pour les 2 pbs), c'est vraiment mineur. |
Fix ecolabdata/ecospheres#74
Depends on opendatateam/udata#2988
Replace #372
Permet d'ajouter une zone géographique à un bouquet lors de la création ou de l'édition d'un bouquet, et de l'afficher dans le récapitulatif et sur la page de détail.
spatial
d'un topic sur data.gouv.fr, donc dépend de Topic: add spatial field udata#2988