Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#547 closed enhancement (fixed)

Passage de /adresses/ vers / ?

Reported by: jocelyn Owned by: sly
Priority: major Component: export cadastre
Keywords: Cc: tyndare, vdct

Description

Serait-il maintenant possible de passer le code de http://cadastre.openstreetmap.fr/adresses/ vers http://cadastre.openstreetmap.fr/ ?

Change History (7)

comment:1 Changed 6 years ago by sly

  • Cc tyndare vdct added

pour !

comment:2 Changed 6 years ago by vdct

pour aussi.
Il faudra assurer une redirection de /adresses/ vers / ?
Je n'ai pas idée des changements dans le code, mais Tyndare maîtrise la chose.

comment:3 Changed 6 years ago by sly

  • Resolution set to fixed
  • Status changed from new to closed

on dirait bien que c'est super bien codé ce que tyndare a fait !
Une simple copie, un petit rewrite et tout marche sans aucun autre changement, trop facile.

si d'aventure j'avais oublié un truc, merci de ré-ouvrir ce ticket et/ou réparer le problème (les modifs sont enregistrées dans git)

comment:4 Changed 6 years ago by jocelyn

Est-ce que le bati ne devrait pas être choisi par défaut au lieu des adresses ?

comment:5 Changed 6 years ago by vdct

Oui, ça paraît logique à cette adresse d'avoir le bâti par défaut.

comment:6 Changed 6 years ago by sly

je me dis que l'interface à suffisamment changée pour qu'un habitué se dise "oulla, ça a changé" et donc "c'est quoi cette case ?"

Pis au pire, il se gourre et il verra bien qu'il n'a pas ce qu'il veut ;-)

Mais j'en ferais pas un fromage, choisissez ce qui vous parait mieux, je vote blanc pour la case par défaut

comment:7 Changed 6 years ago by tyndare

Je pense aussi que le bâti devrais être sélectionné par défaut.
J'ai fait le changement et j'ai mis à jour la règle de rewrite pour que le mode adresses soit quand même sélectionné si l'on vient de /adresses
J'ai aussi déplacé du code javascript dans includes/main.js et le style dans includes/style.css
car si la copie du fichier a marché du premier coup c'est plutôt que j'avais codé comme un bourrin ;-)

Note: See TracTickets for help on using tickets.