Skip to content

Conversation

fabienheureux
Copy link
Member

@fabienheureux fabienheureux commented Jul 10, 2025

Description succincte du problème résolu

Carte Notion/Mattermost/Sentry : https://www.notion.so/accelerateur-transition-ecologique-ademe/Carte-Assistant-Pouvoir-afficher-plusieurs-sous-cat-gories-d-objets-Investigation-15f6523d57d780838ad3ef4fd4807839?source=copy_link

🗺️ contexte: Assistant

💡 quoi: support de plusieurs sous-catégories sur la carte dans l'assistant

🎯 pourquoi: pour améliorer la précision et exhaustivité des points affichés

🤔 comment:

  • Introduction d'une nouvelle manière de passer des données de la vue au template en utilisant un formulaire django plutôt que d'interroger manuellement les paramètres de requête
  • Utilisation uniquement dans la vue carte/produit, à voir pour étendre cette approche plus tard

Exemple résultats / UI / Data

Testé avec Christian sur le vélo, les VAE font remonter des déchetteries

image image

Auto-review

Les trucs à faire avant de demander une review :

  • J'ai bien relu mon code
  • La CI passe bien
  • En cas d'ajout de variable d'environnement, j'ai bien mis à jour le .env.template
  • J'ai ajouté des tests qui couvrent le nouveau code

✅ Reste à faire (PR en cours)

📆 A faire (prochaine PR)

@fabienheureux fabienheureux changed the title wip Gestion de plusieurs sous-catégories sur la carte Jul 10, 2025
@fabienheureux fabienheureux force-pushed the multi-sous-categories branch from d9ad8b7 to be33b4b Compare July 24, 2025 15:27
@fabienheureux fabienheureux added the enhancement New feature or request label Jul 25, 2025
The change moves request data access methods into a new
LegacyMethodsMixin to prepare for future form validation improvements.
@fabienheureux fabienheureux force-pushed the multi-sous-categories branch from 6c0db5e to 12f0a71 Compare July 25, 2025 12:56
@fabienheureux fabienheureux marked this pull request as ready for review July 25, 2025 13:04
@fabienheureux fabienheureux requested a review from a team as a code owner July 25, 2025 13:04
@fabienheureux fabienheureux requested review from kolok and removed request for a team July 25, 2025 13:04
The changes clean up redundant code and simplify request parameter
handling by directly using request.GET instead of a wrapper method.
@kolok
Copy link
Contributor

kolok commented Jul 30, 2025

Hey,
Je veux bien faire la review en pair :)

@fabienheureux
Copy link
Member Author

  • vérifier les query sql pour __sous_categorie__id__in vs appeler directement la table de lien

@fabienheureux
Copy link
Member Author

On attend de fusionner objets et produits/synonymes pour merger cette PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants