Projet

Général

Profil

Development #41103

BAN : permettre de restreindre sur citycode (code INSEE)

Ajouté par Thomas Noël il y a environ 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Thomas Noël
Version cible:
-
Début:
27 mars 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Actuellement on peut filtrer sur postcode mais pour les villes qui ont plusieurs code postaux il serait plus intéressant de filtrer par citycode (code INSEE)

Fichiers

0001-base_adresse-handle-citycode-filter-on-geocoding-end.patch (4,84 ko) 0001-base_adresse-handle-citycode-filter-on-geocoding-end.patch Thomas Noël, 27 mars 2020 14:14
0002-base_adresse-handle-citycode-on-streets-endpoint-411.patch (3,98 ko) 0002-base_adresse-handle-citycode-on-streets-endpoint-411.patch Thomas Noël, 27 mars 2020 16:59
0001-base_adresse-handle-citycode-filter-on-geocoding-end.patch (4,84 ko) 0001-base_adresse-handle-citycode-filter-on-geocoding-end.patch Thomas Noël, 27 mars 2020 16:59

Révisions associées

Révision b5a6d019 (diff)
Ajouté par Thomas Noël il y a environ 4 ans

base_adresse: handle citycode filter on geocoding endpoints (#41103)

Révision 6cdd8fd1 (diff)
Ajouté par Thomas Noël il y a environ 4 ans

base_adresse: handle citycode on streets endpoint (#41103)

Historique

#2

Mis à jour par Thomas Noël il y a environ 4 ans

  • Assigné à mis à Thomas Noël
#3

Mis à jour par Thomas Noël il y a environ 4 ans

#4

Mis à jour par Nicolas Roche il y a environ 4 ans

Moi j'habite là :
https://passerelle.dev.publik.love/base-adresse/test/addresses?q=44%20rue%20charles%20de%20gaulle&citycode=91122

Pour gagner en cohérence, vu que l'attribut citycode est présent dans la classe StreetModel, peut-être compléter aussi le endpoint streets ?
https://passerelle.dev.publik.love/base-adresse/test/streets?citycode=91122

{"err": 1, "err_class": "passerelle.views.WrongParameter", "err_desc": "extra parameters: 'citycode'.", "data": null}

#5

Mis à jour par Thomas Noël il y a environ 4 ans

Nicolas Roche a écrit :

Moi j'habite là :
https://passerelle.dev.publik.love/base-adresse/test/addresses?q=44%20rue%20charles%20de%20gaulle&citycode=91122

Est-ce qu'il y a un soucis ? (si oui, il ne viendra pas de ce patch qui ajoute juste un filtrage)

Pour gagner en cohérence, vu que l'attribut citycode est présent dans la classe StreetModel, peut-être compléter aussi le endpoint streets ?
https://passerelle.dev.publik.love/base-adresse/test/streets?citycode=91122

Bonne idée, je viens d'ajouter un second patch sur la branche.

(J'attache quand même les deux ici, générosité et bienveillance)

#6

Mis à jour par Nicolas Roche il y a environ 4 ans

(non aucun soucis, je vois bien ma rue : ça marche au poil)
je relis...

#7

Mis à jour par Nicolas Roche il y a environ 4 ans

  • Statut changé de Solution proposée à Solution validée
#8

Mis à jour par Thomas Noël il y a environ 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 6cdd8fd1bb77584463f71bf190acc6a757fc1745
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Fri Mar 27 16:57:32 2020 +0100

    base_adresse: handle citycode on streets endpoint (#41103)

commit b5a6d01979a7cad81ee12d180db2579df70c9f14
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Fri Mar 27 14:13:59 2020 +0100

    base_adresse: handle citycode filter on geocoding endpoints (#41103)
#9

Mis à jour par Frédéric Péters il y a environ 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF