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

Rendre champ taxon optionnel pour les observations #400

Open
marcantoinedupre opened this issue Dec 30, 2024 · 1 comment
Open

Rendre champ taxon optionnel pour les observations #400

marcantoinedupre opened this issue Dec 30, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@marcantoinedupre
Copy link
Collaborator

Pour permettre l'ajout de protocole de suivi où les observations ne concernent pas un taxon (par exemple pour les descriptions des horizons de carottage dans le protocole Pédologie des BàO Rhoméo/MhéO - voir p.47 du document) il faudrait pouvoir rendre le champ correspondant complètement optionnel.

Il y a actuellement 2 problèmes identifiés :

  • [frontend] si le champ cd_nom n'est pas affiché dans le formulaire d'une Observation le bouton Valider reste désactivé,
  • [backend] si les données de création reçue contiennent une valeur nulle pour cd_nom (car le champ a été noté en required = false par exemple) une erreur est levée sur le serveur à cause de la contrainte NOT NULL sur ce champ dans la BD.
@marcantoinedupre marcantoinedupre added the enhancement New feature or request label Dec 30, 2024
@camillemonchicourt
Copy link
Member

Je n'en retrouve plus la trace, mais une première étape a été réalisée il me semble dans le cadre de l'ajout des individus (#213).
On en parlait dans un ticket ou une PR mais je ne retrouve pas.
C'est ici il me semble : https://github.com/PnX-SI/GeoNature/pull/3299/files#diff-0d8dccff0edd40b2b02c4089738ea062f4536387f8394c02d5485db4f34f0150R48

De mémoire, avec l'ajout des individus, il s'agit de rendre le champs "cd_nom" de "t_observations" optionnel si le champs "id_individual" est renseigné. Je me pose d'ailleurs un peu des questions sur ce fonctionnement, que je trouve un peu complexe et pas évident, mais c'est un autre sujet.

OK pour aller plus loin et rendre le champs "t_observations.cd_nom" totalement optionnel.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants