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

feat: change 'commune' to 'area' to have different 'geometry' page (#… #591

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

juggler31
Copy link

@juggler31 juggler31 commented Dec 19, 2024

change 'commune' to 'area' to have different 'geometry' page

La fiche commune devient une fiche de zone.

@camillemonchicourt
Copy link
Member

Merci.
Dans le code je parlerai plutôt de "area".
Et quand ça doit être en français, de "zonage" plutôt que de "zone".
A confirmer par @amandine-sahl.

@juggler31
Copy link
Author

Merci. Dans le code je parlerai plutôt de "area". Et quand ça doit être en français, de "zonage" plutôt que de "zone". A confirmer par @amandine-sahl.

Ça marche, par contre il y a déjà beaucoup de chose qui porte le nom "area". si on nomme "area" on ne risque pas de se mélanger ?

@camillemonchicourt
Copy link
Member

OK j'en recause avec @amandine-sahl

@juggler31
Copy link
Author

juggler31 commented Jan 8, 2025

OK j'en recause avec @amandine-sahl

Vous avez pu en parler ensemble ?

@TheoLechemia
Copy link
Member

Bonjour,
Désolé je débarque un peu après la bataille.

Ça marche, par contre il y a déjà beaucoup de chose qui porte le nom "area".

Oui il y a déjà la notion d'areas, et c'est sur celle la qu'il faut s'appuyer pour les "zonages". GeoNature possède déjà une table des zonages et des type de zonage, nommée respectivement ref_geo.l_areas et ref_geo.bib_area_type, dans GeoNature-atlas, ces tables sont mappé en vue materialisées : vm_l_areas et vm_bib_areas_types (https://github.com/PnX-SI/GeoNature-atlas/blob/master/atlas/modeles/entities/vmAreas.py#L13-L42). Il faut s'appuyer sur celle la plutôt que de recréer une table des zonages.
Par conter l'installation de ces VM est dans un fichier à part https://github.com/PnX-SI/GeoNature-atlas/blob/master/data/atlas_with_extended_areas.sql . A clarifier et à integrer dans l'installation "normal" pour moi

@camillemonchicourt
Copy link
Member

Oui bien vu, d'ailleurs ce script atlas_with_extended_areas.sql est mentionné et nécessaire pour le fonctionnement existant par zone : #209 (comment)

Donc en effet il faut intégrer ça de base et s'appuyer dessus sans recréer des tables et modèles similaires.

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