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

POC : Multiple variables with suggesters #900

Merged
merged 3 commits into from
Mar 8, 2024

Conversation

Grafikart
Copy link
Collaborator

@Grafikart Grafikart commented Mar 6, 2024

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

@Grafikart Grafikart changed the title POC : Multiple variable with suggesters POC : Multiple variables with suggesters Mar 6, 2024
@nsenave
Copy link
Contributor

nsenave commented Mar 7, 2024

La modélisation me va a priori 👍

@romaintailhurat
Copy link
Collaborator

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
Copy link
Contributor

nsenave commented Mar 7, 2024

@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.

@Grafikart Grafikart force-pushed the feat/optionResponses branch from 618a94c to a5cb73e Compare March 8, 2024 12:38
Copy link

sonarqubecloud bot commented Mar 8, 2024

Quality Gate Failed Quality Gate failed

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

See analysis details on SonarCloud

@ddecrulle ddecrulle merged commit 812b798 into ref/components Mar 8, 2024
6 of 7 checks passed
@ddecrulle ddecrulle deleted the feat/optionResponses branch March 8, 2024 12:51
@ddecrulle
Copy link
Contributor

SUGGESTER_${RESPONSE_NAME}_${ATTRIBUTE_NAME_IN_NOMENCLATURE}

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

Successfully merging this pull request may close these issues.

4 participants