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

[SYNTHESE] Maintien du lien lors de la désactivation des profils de taxons #3311

Closed
MathieuLagarde opened this issue Jan 10, 2025 · 9 comments

Comments

@MathieuLagarde
Copy link

Version
2.15.0

Description du bug
Après désactivation de l'affichage des informations liées aux profils de taxons, le lien sur le nom du taxon est conservé et actif.

Comportement attendu
Ne plus avoir de lien sur le nom du taxon?

Comment reproduire
Désactiver le profil de taxon (ajout de l'option ENABLE_PROFILES = false dans la partie [FRONTEND] du fichier config geonature_config.toml)

@camillemonchicourt camillemonchicourt added this to the 2.15.1 milestone Jan 10, 2025
@edelclaux
Copy link
Contributor

Salut,
ESt-ce que le lien dont tu parles est bien celui la ?
image

@camillemonchicourt
Copy link
Member

Oui c'est bien ce lien.
Mais en fait c'est normal que ce lien reste actif quand on désactive les profils.
On peut désactiver les profils, mais quand même vouloir la fiche Taxon (depuis qu'elle a un rôle élargi avec la 2.15).
La lien doit se désactiver quand on désactive les fiches taxon.
Mais en fait on ne peut pas désactiver cette fiche taxon ?
Seulement désactiver certains de ses onglets ? https://github.com/PnX-SI/GeoNature/blob/master/config/default_config.toml.example#L444-L449

@camillemonchicourt camillemonchicourt removed this from the 2.15.1 milestone Jan 10, 2025
@edelclaux
Copy link
Contributor

edelclaux commented Jan 10, 2025

Tout à fait.
Dans la première mouture, j'avais proposé de rendre les fiches taxons désactivbles, mais il avait été choisit de minimiser la config, et de ne pas rendre la page optionnelle.

Edit: ah non, ça n'a jamais été désactivable. Autant pour moi.

La fiche profile se trouve maintenant derrière l'onglet profile de la fiche espèce. Ce dernier devrait normalement ne pas apparaitre si les fiches profiles sont désactivées.
image

Est-ce que c'est bien le cas ?

Est-ce qu'on souhaite pouvoir désactiver la fiche taxon ?

@MathieuLagarde
Copy link
Author

Merci pour vos réponses.
Oui, effectivement, il s'agit bien de ce lien.
Et c'est effectivement toute la fiche taxon que je souhaiterais pouvoir désactiver, en rendant le lien inactif.
Cette fiche fait peu ou prou doublon avec les fiches taxons de l'Atlas, apportant une certaine confusion dans la navigation de la plateforme (pourquoi avoir la même information à deux endroits différents ???). D'autant que les indicateurs mentionnés entre la fiche taxon de l'atlas et la fiche taxon de la synthèse diffèrent.
ça me semble vraiment nécessaire de pouvoir désactiver cette fiche taxon de la synthèse.

@camillemonchicourt
Copy link
Member

Oui, la similitude avec l'Atlas pose question et a un peu été débattue dans le sujet initial : #2981 (comment)

Mais certains concentrent leur activité sur GeoNature et pour toutes les personnes qui saisissent ou utilisent GeoNature (disposant ou non d'un GeoNature-atlas), il est intéressant d'avoir des fiches taxons directement dans GeoNature. Et les 2 outils peuvent s'adresser à des utilisateurs et publics différents.
Mais la duplication de pas mal de choses et sa maintenance à moyen/long terme peut en effet poser question.

Dans tous les cas, les fiches taxons étaient déjà accessibles auparavant, comme on le voit dans votre instance :

image

Exemple : https://data.biodiversite-bretagne.fr/geonature/#/synthese/taxon/2623

Elles ont été enrichies avec la 2.15 (passant d'une fiche profil à une fiche taxon) et un lien a été ajouté directement dans la liste des observations de la Synthèse.

Mais OK, on va voir pour ajouter rapidement dans la 2.15.2 un paramètre permettant de les désactiver complétement.
Et on va en profiter pour clarifier et mettre en cohérence les paramètres qui ont des noms proches :

@edelclaux
Copy link
Contributor

edelclaux commented Jan 10, 2025

Je me saisis du dev.

  1. Ajout de l'option ENABLE_TAXON_SHEET dans la config, dans la section [FRONTEND] (au même niveau que ENABLE_PROFILES).
    Utilisation pour:
    • afficher ou non la fiche taxon
    • activer ou non l'hyperlien dans la synthèse
    • activer ou non le bouton dans la fiche observation

Une alternative serait de mettre cette option dans la section [SYNTHESE]

  1. Suppression de l'option SYNTHESE.TAXON_SHEET.ENABLE_PROFILE et utilisation de l'option FRONTEND.ENABLE_PROFILE à la place

Une alternative serait d'utiliser l'option FRONTEND.ENABLE_PROFILE pour créer l'option SYNTHESE.TAXON_SHEET.ENABLE_PROFILE, par soucis de consistance avec les autres onglets de la fiche taxon.

Ca vous semble ok ?

@camillemonchicourt
Copy link
Member

camillemonchicourt commented Jan 10, 2025

OK merci.

  • ENABLE_TAXON_SHEET > ENABLE_TAXON_SHEETS et je pense que ces fiches taxons seront à terme accessibles ailleurs que depuis la Synthèse, donc à ne pas mettre dans cette section. Si à FALSE, alors l'URL des fiches taxons e doit pas être accessible.
  • Si FALSE, il faut aussi virer le lien sur les formulaires Occtax (et le renommer comme il faut en passant : "Voir la fiche du taxon")
    image.
  • On peut vouloir masquer les profils sur les fiches taxon (car actuellement ils permettent d'accéder à des données sensibles qui peuvent être floutées par ailleurs), mais garder la fonctionnalité des profils par ailleurs (Calcul et affichage des notes /3 des observations pour la validation, message d'alerte dans la saisie Occtax...).
  • Je partirai bien sur du ENABLE_PROFILE_TAB, ENABLE_TAXONOMY_TAB.

@jacquesfize
Copy link
Contributor

Effectué dans la version 2.15.2 de GeoNature

@camillemonchicourt
Copy link
Member

Corrigé et amélioré dans la 2.15.2 avec les paramètres ENABLE_TAXON_SHEETS et SYNTHESE.TAXON_SHEET.ENABLE_TAB_PROFILE : https://github.com/PnX-SI/GeoNature/blob/2.15.2/config/default_config.toml.example#L444-L451.

Et mise en cohérence du paramètre global ENABLE_PROFILES (si il est à FALSE alors l'onglet PROFIL ne s'affiche pas dans les fiches taxons, même si leur paramètre est à TRUE)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants