base_adresse: rendre les rues non sélectionnables, ne permettre que les adresses complètes avec numéro de rue (#76376) #207

Merged
bdauvergne merged 1 commits from wip/76376-base-adresse-forcer-la-recherche into main 2023-05-15 10:36:38 +02:00
Owner
No description provided.
bdauvergne changed title from WIP: base_adresse: rendre les rues non sélectionnable, ne permettre que les adresses complètes avec numéro de rue (#76376) to WIP: base_adresse: rendre les rues non sélectionnables, ne permettre que les adresses complètes avec numéro de rue (#76376) 2023-04-14 16:54:47 +02:00
bdauvergne force-pushed wip/76376-base-adresse-forcer-la-recherche from 6f3a9685d4 to 3f981e9adc 2023-04-14 17:15:23 +02:00 Compare
bdauvergne force-pushed wip/76376-base-adresse-forcer-la-recherche from 3f981e9adc to c9092de6ec 2023-04-17 07:38:54 +02:00 Compare
bdauvergne changed title from WIP: base_adresse: rendre les rues non sélectionnables, ne permettre que les adresses complètes avec numéro de rue (#76376) to base_adresse: rendre les rues non sélectionnables, ne permettre que les adresses complètes avec numéro de rue (#76376) 2023-04-17 07:49:36 +02:00
bdauvergne force-pushed wip/76376-base-adresse-forcer-la-recherche from c9092de6ec to 0359929bfd 2023-04-17 08:12:25 +02:00 Compare
tnoel requested changes 2023-04-20 17:31:21 +02:00
tnoel left a comment
Owner

Je n'arrive pas à être à l'aise avec cette astuce... un coup je trouve ça bien, le lendemain je m'inquiète de l'API mouvante de la BAN.

Je me dis que dans un premier temps on pourrait juste envoyer le type= à leur API, et retourner le résultat, sans intelligence. Et voir si ça marche pas tout bêtement ainsi chez nos clients avisés.

Sinon, il reste un "print" que je signale donc ici, mais je veux bien un autre oeil.

Je n'arrive pas à être à l'aise avec cette astuce... un coup je trouve ça bien, le lendemain je m'inquiète de l'API mouvante de la BAN. Je me dis que dans un premier temps on pourrait juste envoyer le type= à leur API, et retourner le résultat, sans intelligence. Et voir si ça marche pas tout bêtement ainsi chez nos clients avisés. Sinon, il reste un "print" que je signale donc ici, mais je veux bien un autre oeil.
@ -179,1 +195,4 @@
address.update_timestamp()
if features and no_housenumber and disable_streets:
result.insert(0, {'id': 'X', 'text': _('Please provide a house number'), 'children': []})
print(no_housenumber, result)
Owner

bip !

bip !
bdauvergne force-pushed wip/76376-base-adresse-forcer-la-recherche from 0359929bfd to 62d88170a1 2023-04-21 02:07:20 +02:00 Compare
Author
Owner

Je me dis que dans un premier temps on pourrait juste envoyer le type= à leur API, et retourner le résultat, sans intelligence. Et voir si ça marche pas tout bêtement ainsi chez nos clients avisés.

Ok, mais je ne trouve pas ça compréhensible pour les gens, les clients je m'en fous un peu.

> Je me dis que dans un premier temps on pourrait juste envoyer le type= à leur API, et retourner le résultat, sans intelligence. Et voir si ça marche pas tout bêtement ainsi chez nos clients avisés. Ok, mais je ne trouve pas ça compréhensible pour les gens, les clients je m'en fous un peu.
bdauvergne force-pushed wip/76376-base-adresse-forcer-la-recherche from 62d88170a1 to 17ad2ae37d 2023-04-21 02:21:09 +02:00 Compare
bdauvergne requested review from tnoel 2023-04-21 02:21:20 +02:00
Author
Owner

Voilà je suis revenu au code le plus simple possible, on valide type parmi les 4 choix possibles et on le passe l'API BAN.

Voilà je suis revenu au code le plus simple possible, on valide type parmi les 4 choix possibles et on le passe l'API BAN.
tnoel approved these changes 2023-05-09 12:12:38 +02:00
bdauvergne merged commit 84cd51957e into main 2023-05-15 10:36:38 +02:00
bdauvergne deleted branch wip/76376-base-adresse-forcer-la-recherche 2023-05-15 10:36:38 +02:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 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/passerelle#207
No description provided.