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

le retour de bulk_launch_uuid #276

Merged
merged 3 commits into from
Nov 28, 2023
Merged

le retour de bulk_launch_uuid #276

merged 3 commits into from
Nov 28, 2023

Conversation

fchabouis
Copy link
Collaborator

J'avais pas bien expliqué le but de ce champ : permettre de savoir que plusieurs imports viennent de la même commande (typiquement un import france qui génère plusieurs imports départementaux). Je remets le code comme avant et je rajoute des tests sur cet aspect là.

@fchabouis fchabouis changed the base branch from main to the_import November 27, 2023 11:40
Copy link
Contributor

@pauletienney pauletienney left a comment

Choose a reason for hiding this comment

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

Je me demande si ce n'est pas le moment de réfléchir à un mécanisme pour que l'on oublie pas de générer un BuildingImport si on devait écrire de nouveaux scripts d'import.

app/batid/services/imports/import_bdtopo.py Show resolved Hide resolved
@pauletienney pauletienney self-requested a review November 27, 2023 16:35
@pauletienney pauletienney merged commit 7025af1 into the_import Nov 28, 2023
1 check passed
@pauletienney pauletienney deleted the bulk_launch_uuid branch November 28, 2023 08:49
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.

2 participants