Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Identifiants ZFE : ajout syndicats pour Tours et Lyon #4415

Merged
merged 1 commit into from
Jan 17, 2025

Conversation

AntoineAugusti
Copy link
Member

@AntoineAugusti AntoineAugusti commented Jan 15, 2025

La ZFE de Tours est rattachée à l'AOM qui est un syndicat et non la métropole. On a la même chose pour la ZFE des voies spéciales de Lyon.

Les données ne correspondant pas, le zfe_id est vide pour cette ZFE.

Je mets à jour les données immédiatement et je réfléchis à une meilleure solution long terme pour identifier ces données à corriger.

Voir sur Front

@AntoineAugusti AntoineAugusti requested a review from a team as a code owner January 15, 2025 08:13
@AntoineAugusti AntoineAugusti changed the title ZFE : mise à jour entité responsable pour Tours Identifiants ZFE : ajout syndicats pour Tours et Lyon Jan 15, 2025
@@ -40,5 +40,5 @@ siren;code;epci_principal;autres_siren
246700488;STRASBOURG;Eurométropole de Strasbourg;
248300543;TOULON;Métropole Toulon Provence Méditerranée;
253100986;TOULOUSE;Toulouse métropole;243100518
243700754;TOURS;Tours Métropole Val de Loire;
243700754;TOURS;Tours Métropole Val de Loire;200085108
Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -19,7 +19,7 @@ siren;code;epci_principal;autres_siren
247200132;LE MANS;Le Mans Métropole;
200093201;LILLE;Métropole Européenne de Lille;
248719312;LIMOGES;Limoges Métropole;
200046977;LYON;Métropole de Lyon;
200046977;LYON;Métropole de Lyon;256900994
Copy link
Member Author

Choose a reason for hiding this comment

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

@AntoineAugusti
Copy link
Member Author

Le code qui permet de renseigner l'identifiant d'une ZFE.

def zfe_id(siren) do
zfe_id =
CSVDocuments.zfe_ids()
|> Enum.find_value(&if siren == &1["siren"] or String.contains?(&1["autres_siren"], siren), do: &1["code"])
if is_nil(zfe_id) do
Logger.error("Could not find zfe_id for SIREN #{siren}")
end
zfe_id
end

@AntoineAugusti AntoineAugusti added this pull request to the merge queue Jan 17, 2025
Copy link
Contributor

@thbar thbar left a comment

Choose a reason for hiding this comment

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

Ça semble tout bon (revue juste sur le code). Merci pour le lien fourni sur le code.

Merged via the queue into master with commit 4897113 Jan 17, 2025
4 checks passed
@AntoineAugusti AntoineAugusti deleted the zfe_ids_tours_syndicat branch January 17, 2025 10:33
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.

3 participants