-
Notifications
You must be signed in to change notification settings - Fork 15
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
Nichoir #46
base: master
Are you sure you want to change the base?
Nichoir #46
Conversation
suppression case occupation
This reverts commit 0127ea5.
…otocoles_suivi into dev_nichoir
…otocoles_suivi into dev_nichoir
Pourriez vous déindexer le fichier : |
"required": true | ||
}, | ||
"observers": { | ||
"type_widget": "datalist", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pourquoi vous redéfinissez le champs "observers" avec un composant "datalist", alors que celui-ci est déjà géré nativement et par défaut au niveau des visites ?
A éviter il me semble.
], | ||
"uuid_field_name": "uuid_observation", | ||
"specific": { | ||
"uuid_observation": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je ne comprends pas pourquoi vous redéfinissez le champs UUID qui est déjà présent dans la BDD et peuple automatiquement ?
"uuid_observation": { | ||
"attribut_label": "uuid" | ||
}, | ||
"id_observation": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je ne comprends pas non plus pourquoi vous redéfinissez les champs d'ID ?
Vous les affichez et les rendez saisissables par les utilisateurs ?
Cela ne me semble pas souhaitable car c'est un champs caché et peuplé automatiquement par la BDD
"hidden": false, | ||
"required": false | ||
}, | ||
"medias": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je ne comprends pas pourquoi les médias sont redéfinis alors qu'ils sont gérés nativement dans le module ?
"attribut_label": "Id visite", | ||
"hidden": true | ||
}, | ||
"cd_nom": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je ne comprends pas pourquoi le champs "cd_nom" est redéfini alors qu'il est géré nativement dans le module ?
}, | ||
"hidden": true | ||
}, | ||
"base_site_code": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pourquoi redéfinir ce champs, géré nativement par le module ?
"type_widget": "textarea", | ||
"attribut_label": "Remarque" | ||
}, | ||
"medias": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pourquoi redéfinir ce champs géré nativement par le module ?
Création du module Nichoir
Modules fonctionnels avec synchronisation des données sur gn_synthèses.
Pas d'extraction des données, mais création de requête personnalisée pour exploiter les données (sur QGIS, voir Lizmap)
Certains champs sont successibles d'évoluer en fonction des utilisations :
pose_localisation
qui liste les différents supports de pose de nichoir etnichoir_type
qui liste les types de nichoirs mis en placeVoir #45