Skip to content

Admin: Recalculer automatiquement le geocoding des organisations, sauf cas particuliers [GEN-2023]#4910

Merged
tonial merged 1 commit intomasterfrom
alaurent/geocoding
Nov 14, 2024
Merged

Admin: Recalculer automatiquement le geocoding des organisations, sauf cas particuliers [GEN-2023]#4910
tonial merged 1 commit intomasterfrom
alaurent/geocoding

Conversation

@tonial
Copy link
Contributor

@tonial tonial commented Oct 10, 2024

🤔 Pourquoi ?

Reste à savoir qui va taguer les organisations dont les adresses donnent un mauvais geocoding 🙄

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@tonial tonial added the modifié Modifié dans le changelog. label Oct 10, 2024
@tonial tonial self-assigned this Oct 10, 2024
@tonial tonial force-pushed the alaurent/geocoding branch 3 times, most recently from aa5bf96 to 5ed57fe Compare October 10, 2024 09:56
Copy link
Contributor

@rsebille rsebille left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je garderais le champ actuel pour laisser les gens demander un recalcul instantanément mais pour la correction du stock je verrais plus vu un script comme on avais fait pour les candidats (update_job_seeker_coords) et les entreprises (update_companies_coords), quitte a le lancer ensuite via une tâche cron.

@francoisfreitag francoisfreitag changed the title Admin: Recalculer automatiquement le geocoding des organisations , sauf cas particuliers GEN-2023 Admin: Recalculer automatiquement le geocoding des organisations, sauf cas particuliers [GEN-2023] Oct 10, 2024
@tonial tonial force-pushed the alaurent/geocoding branch from 5ed57fe to 11ddac9 Compare October 10, 2024 14:06
@tonial
Copy link
Contributor Author

tonial commented Oct 11, 2024

Ça vaut sans doute le coup d'en discuter au prochain SU pour être bien en phase sur ce qu'on veux faire : j'ai l'impression qu'on n'a pas compris la carte notion pareil

@tonial tonial force-pushed the alaurent/geocoding branch 2 times, most recently from e5d7ec4 to 7a9a481 Compare October 17, 2024 14:14
@rsebille
Copy link
Contributor

Au final on c'est pas capté :(.
La PR répond à ce qui est demandé dans la carte notion mais on se retrouve à soigner 1 symptôme précis et pas le problème général, pour ça que je parlais des scripts update_*_coords qui avec un peu de modification pourraient être lancés en cron et venir rattraper ce genre d'oublis tout en faisant la reprise de stock au passage, et ça resterais plus ou moins d'actualité une fois l'autocomplétion BAN ajouté pour les entreprises et organisations prescriptrice (comme pour les utilisateurs) avec le projet "Adresse".

@tonial tonial force-pushed the alaurent/geocoding branch from 7a9a481 to 2b8dd1c Compare November 14, 2024 08:57
@tonial tonial requested a review from rsebille November 14, 2024 08:57
@tonial
Copy link
Contributor Author

tonial commented Nov 14, 2024

Après discussion orale, la solution idéale serait de détecter un changement de géolocalisation, et afficher une modale pour proposer de l'appliquer, mais que c'est beaucoup trop complexe à faire dans l'admin.
La solution proposée dans cette PR est plus simple, et permettra de voir dans quelques semaines/mois si on a vraiment des cas où l'on souhaite ne pas appliquer le re-calcul automatique.

Comment on lines +48 to +50
verbose_name="recalculer le geocoding",
help_text="Si cette case est cochée, les coordonnées géographiques seront mises à jour si l'adresse est "
"correctement renseignée.",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je sais pas comment le formuler correctement mais peut-être spécifier que ce comportement est seulement pour l'admin, histoire de ne pas induire en erreur si quelqu'un tombe sur ce champ.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Si cette case est cochée, les coordonnées géographiques seront mises à jour si l'adresse est correctement renseignée dans le formulaire d'admin.

C'est mieux ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je pense 😁, toutes les formulations me font bizarre donc si toi ça te semble clair let's go.

@tonial tonial force-pushed the alaurent/geocoding branch 2 times, most recently from eb9cf61 to 962b235 Compare November 14, 2024 10:53
We want to always update except for specific organizations where the
geocoding is known to give bad results
@tonial tonial force-pushed the alaurent/geocoding branch from 962b235 to 8547b3f Compare November 14, 2024 13:31
@tonial tonial enabled auto-merge November 14, 2024 13:32
@tonial tonial added this pull request to the merge queue Nov 14, 2024
Merged via the queue into master with commit 59776b3 Nov 14, 2024
@tonial tonial deleted the alaurent/geocoding branch November 14, 2024 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants