Skip to content

fix: remove default country from food#2003

Open
vjousse wants to merge 12 commits intomasterfrom
fix/food-default-country
Open

fix: remove default country from food#2003
vjousse wants to merge 12 commits intomasterfrom
fix/food-default-country

Conversation

@vjousse
Copy link
Copy Markdown
Collaborator

@vjousse vjousse commented Apr 1, 2026

🔧 Problem

fixes #1975

@vjousse vjousse self-assigned this Apr 1, 2026
@vjousse vjousse force-pushed the fix/food-default-country branch from ef2c52a to 0d17113 Compare April 1, 2026 16:01
@vjousse vjousse force-pushed the fix/food-default-country branch from 430f672 to fe0d890 Compare April 2, 2026 15:05
@camcoq-start
Copy link
Copy Markdown
Collaborator

camcoq-start commented Apr 3, 2026

@vjousse

  1. pour le 'haricot vert par défaut (2025)', on attend 18000 km d'avion.
    Si je clique sur le bateau, on attend 18000 km de bateau réfrigéré.

cf: capture de le review app où on a 0km
image

  1. pour la farine UE, cet ingrédient a un transport réfrigéré de la transformation à la France. Les 160km de la matière vers la transformation sont sans réfrigéré.
    Donc c'est 3000km de camion réfrigéré et 160km de camion non réfrigéré
    cf: capture de le review app où on a 2500 km de camion réfrigéré et 660 non réfrigéré
image

A part ces 2 tests, tous les autres sont bons.
Je n'ai pas noté de régression sur les calculettes textile et objet par rapport aux distances et aux scores.

Il reste à tester les exemples food qui eux peuvent changer avec les nouvelles distances.

Vraiment top sinon !

@vjousse
Copy link
Copy Markdown
Collaborator Author

vjousse commented Apr 3, 2026

@camcoq-start le problème avec le haricot devrait être réglé. Par contre pour la Farine, le comportement de cette PR est le même comportement que sur l’alimentaire actuel, à quel changement tu fais référence ? (quelle est l’issue qui s’y rapporte ?)

@camcoq-start
Copy link
Copy Markdown
Collaborator

@vjousse pour l'avion c'est réglé.
Pour le bateau par contre j'ai 0km lorsque je le sélectionne.
On attend: 18 000km de bateau lorsqu'on 'désactive' le mode avion, qui s'ajoute au transport routier
image

Pour ma remarque sur le réfrigéré, c'est à ignorer. C'est un autre écart par rapport à la documentation qu'on peut pour l'instant ne pas corriger.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

(Transport) Mettre en cohérence les distances depuis la "France" et "(par défaut) France"

2 participants