InseeFr / Lunatic

Library of questionnaire components
https://inseefr.github.io/Lunatic/
MIT License
20 stars 22 forks source link

POC : Multiple variables with suggesters #900

Closed Grafikart closed 8 months ago

Grafikart commented 8 months ago

POC

Démonstration d'une solution possible pour le cas d'un suggester qui permet de sélectionner plusieurs infos.

Modélisation

On ajouterais une propriété "optionResponses" au JSON qui permettrait d'associer une propriété du référentiel à une variable dans Lunatic.

    "optionResponses": [{
        "name": "PRODUCT_NAME",
        "attribute": "label"
    },{
        "name": "PRODUCT_PRICE",
        "attribute": "price"
    }],

La clef "response" serait toujours utilisé pour sauvegardé l'id sélectionnée.

Story

La nouvelle story est visible via http://localhost:9999/?path=/story/components-suggester--option-responses

Related issues

nsenave commented 8 months ago

La modélisation me va a priori 👍

romaintailhurat commented 8 months ago

Une suggestion sur le nommage des variables : pour éviter des collisions de nommage, on devrait adopter un schéma de nommage, du style SUGGESTER_<ID de la liste>_<nom de la clé>

Par exemple SUGGESTER_PRODUCT_LABEL pour le premier exemple.

nsenave commented 8 months ago

@romaintailhurat ça me paraît judicieux aussi, mais si on veut forcer ces conventions de nommage, il faut le faire sur l'outil de génération (Pogues) mais pas dans Lunatic pour ne pas l'encombrer à mon sens.

Ceci dit si les tests/exemples qui figurent dans Lunatic suivent la convention qu'on souhaite c'est un plus.

sonarcloud[bot] commented 8 months ago

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

ddecrulle commented 8 months ago

SUGGESTER_${RESPONSE_NAME}_${ATTRIBUTE_NAME_IN_NOMENCLATURE}