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] Utilisation du paramètre SYNTHESE.LIST_COLUMNS_FRONTEND pour personnaliser les colonnes #2749

Closed
mvergez opened this issue Oct 11, 2023 · 1 comment

Comments

@mvergez
Copy link
Contributor

mvergez commented Oct 11, 2023

Bonjour,

Dans le cadre d'une prestation pour l'Agence Régionale de la Biodiversité en île de France, je me suis aperçu que le paramètre frontend SYNTHESE.LIST_COLUMNS_FRONTEND ne fonctionnait pas comme prévu.

En effet, j'ai voulu ajouter la colonne meta_update_date dans le paramètre LIST_COLUMNS_FRONTEND dans le fichier geonature_config.toml. Cette colonne existe bien dans la vue v_synthese_for_web_app. Or la colonne est vide en frontend.

Cela vient de ces lignes :

columns = [
"id",
VSyntheseForWebApp.id_synthese,
"date_min",
VSyntheseForWebApp.date_min,
"lb_nom",
VSyntheseForWebApp.lb_nom,
"cd_nom",
VSyntheseForWebApp.cd_nom,
"observers",
VSyntheseForWebApp.observers,
"dataset_name",
VSyntheseForWebApp.dataset_name,
"url_source",
VSyntheseForWebApp.url_source,
"unique_id_sinp",
VSyntheseForWebApp.unique_id_sinp,
"nom_vern_or_lb_nom",
nom_vern_or_lb_nom,
"count_min_max",
count_min_max,
"entity_source_pk_value",
VSyntheseForWebApp.entity_source_pk_value,
]

Qui remonte uniquement ces colonnes de la vue. En ajoutant manuellement meta_update_date dans cette liste cela fonctionné.

Cela me fait me poser 2 questions :

  1. Est-il possible de charger le paramètre LIST_COLUMNS_FRONTEND pour ajouter les colonnes demandées ?
  2. Y a-t-il pas trop de colonnes remontées qui ne sont pas utilisées par le frontend ?

Je propose de traiter le premier point ci-dessus, est-ce que cela vous convient ?

Merci d'avance pour vos retours !

@camillemonchicourt
Copy link
Member

Oui le principe est que l'on puisse customiser et afficher ou non des champs différents de ceux proposés par défaut, à partir des champs présents dans la vue v_synthese_for_web_app.
Donc si on ne peut pas prendre les champs qu'on veut dans cette vue, ce n'est pas souhaitable.

Sauf si le fait d'ajouter des champs dans la route /for_web va alourdir la réponse de cette API pour tous, en renvoyant dans tous les cas tous les champs listés, tout le temps, même si ils ne sont pas demandés dans la configuration des colonnes à afficher.

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

No branches or pull requests

3 participants