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

Ajout des statuts de protection aux fiches espèces #563

Open
wants to merge 16 commits into
base: develop
Choose a base branch
from

Conversation

pbarille
Copy link

Ajout des statuts de protection des espèces aux fiches espèces :

image

L'affichage des statuts se base sur la BDC présente dans la base geonature2db (via foreign data wrapping et materialized view)

@camillemonchicourt
Copy link
Member

OK merci. Intéressant.

  • Il faudra voir si cela fonctionne aussi et comment pour ceux qui installent GN-atlas sans GN
  • Ne pas mettre des noms d'utilisateurs en dur dans les scripts (geonatadmin, geonatatlas, geonaturedbserver, ...) qui peuvent être différents selon les configurations
  • Voir comment gérer la mise à jour des instances déjà existantes

@amandine-sahl
Copy link
Contributor

Merci pour la contribution mais telle qu'elle est implémentée, elle ne peut pas s'appliquer aux différents territoires utilisant Geonature-atlas.

  • Les filtres geographiques utilisés limitent la requête à la bretagne
  • Les types de status LRNh, LRNm, LRNn, LRNp, LRRm , LRRn n'existent nativement pas la la bdc_status

Par ailleurs pourquoi ne pas avoir utilisée directement la vue v_bdc_statut?

@MathieuLagarde
Copy link

J'ai créé les cd_type_statut LRNh, LRNm, LRNn, LRNp, LRRm , LRRn pour pouvoir distinguer facilement les listes rouges des oiseaux nicheurs, migrateurs, de passage, hivernant, qui, autrement ne sont pas différenciables. Certes, pour la liste rouge nationale, il y a une valeur présente dans le champ rq_statut, mais cette info est mêlée à d'autres. Ce n'est peut-être pas la solution la plus satisfaisante, mais c'est celle que j'ai trouvé la plus simple à mettre en place sur le moment.

Concernant l'utilisation de la vue v_bdc_statut, je ne pourrais pas répondre, nous avons fait appel à un prestataire pour développer l'affichage de ces statuts.

@camillemonchicourt
Copy link
Member

OK mais donc cette contribution est spécifique à différents niveaux. Si on l'intègre en l'état elle ne fonctionnera pas pour les autres utilisateurs et ferait planter leur GeoNature-atlas.
Il est donc nécessaire de rendre cette évolution générique pour pouvoir l'intégrer dans GeoNature-atlas.

@pbarille
Copy link
Author

Bonjour,

Merci pour ces retours.

  • Il faudra voir si cela fonctionne aussi et comment pour ceux qui installent GN-atlas sans GN

    • J'ai ajouté un if $use_ref_geo_gn2 pour l'exécution de la requête sql faisant le lien avec geonature dans install_db.sh pour qu'il n'y ait pas de problème pour les utilisateurs d'atlas n'ayant pas de geonature
  • Ne pas mettre des noms d'utilisateurs en dur dans les scripts (geonatadmin, geonatatlas, geonaturedbserver, ...) qui peuvent être différents selon les configurations

    • C'est corrigé pour geonatadmin et geonatatlas, en revanche geonaturedbserver j'ai l'impression que c'est un paramètre, non ?
  • Voir comment gérer la mise à jour des instances déjà existantes

    • Il faudrait ajouter quelquechose dans les scripts sql du dossier update et l'indiquer dans les notes de version j'imagine ?
  • Les filtres geographiques utilisés limitent la requête à la bretagne

    • C'est corrigé dans la requête par défaut et nous avons ajouté un paramètre pour pouvoir le personnaliser si besoin
  • Les types de status LRNh, LRNm, LRNn, LRNp, LRRm , LRRn n'existent nativement pas la la bdc_status

    • C'est corrigé dans la requête avec les statuts par défaut de la BDC et nous avons ajouté un paramètre pour pouvoir le personnaliser si besoin
  • pourquoi ne pas avoir utilisée directement la vue v_bdc_statut?

    • Nous avons un champ qui diffère entre notre développement et la vue, cela nous semble cependant possible de nous baser sur la vue (via le champ cd_sig mais qui est moins explicite). Nous imaginons également que les développements de notre côté avaient été initiés avant que cette vue soit déployée.

D'avance merci pour vos retours.
Bonne journée

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.

None yet

4 participants