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

Clefs primaires des territoires -> clefs naturelles #652

Merged
merged 28 commits into from
Nov 5, 2024

Conversation

alexisig
Copy link
Collaborator

@alexisig alexisig commented Oct 16, 2024

  • Remplacement du lien Project.land_id -> Land.id (id autoincrémenté) par Project.land_id -> Land.official_id (id naturelle)
  • Même chose pour le champs Project.look_a_like
  • Déplacement de tous les modèles finaux dans un schema dédié nommé "for_app"
  • Retrait de Sqlfluff -> bon linter / fixer, mais trop lent
  • Prise en compte totale des entités de l'admin express pour les regions, départements et epcis, et leurs liens entre eux
  • Ajout des EPT en tant qu'EPCI
  • Ajout d'un model consommation_cog_2024 qui est le resultat des données de consommation 2023 découpés au COG 2024
  • Ajout des SCOT depuis l'API ALN
  • Ajout de nombreux tests sur les données finales / de staging
  • Suppression des projets existants liés à des SCOT -> il était impossible de les lier faute d'identifiant officiel lors du premier chargement des SCOT en base
  • Ajout d'une fonction init_unmanaged_schema_for_tests permettant de faire des tests en utilisant des models unmanaged (les models crées par airflow)
  • Mise à jour des méthodes de query des territoires : ils devraient être queryable par leur clef naturelle, et non plus par leur id autoincrementé

Message affiché dans le cas de fusion de commune
image


Message affiché dans le cas de division de commune
image


Message affiché en cas d'absence de donnée
image

@alexisig alexisig changed the base branch from master to staging October 16, 2024 21:07
@alexisig alexisig force-pushed the fix-natural-keys branch 2 times, most recently from 7a7b803 to 91c909b Compare October 24, 2024 14:34
@alexisig alexisig marked this pull request as ready for review October 29, 2024 09:32
Copy link

sonarqubecloud bot commented Nov 5, 2024

@alexisig alexisig merged commit 1105358 into staging Nov 5, 2024
8 checks passed
@alexisig alexisig linked an issue Nov 5, 2024 that may be closed by this pull request
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.

BUG: La recherche par code INSEE génère une erreur 500
1 participant