fix: add VariableReference in variable for ucq - #185
Conversation
nsenave
left a comment
nsenave
left a comment
There was a problem hiding this comment.
Retour sur la doc de 'VariableReference' dans l'objet vairble
| VariableReference indicates the collected variable whose values | ||
| dynamically define the response options when the question | ||
| choiceType is set to VARIABLE. | ||
| It must not be used when choiceType is CODE_LIST. |
There was a problem hiding this comment.
C'est un peu bizarre de parler de "cette question est de type 'liste de code'" dans un objet variable 🤔
Après j'avais pas conscience (ou plus en mémoire) qu'on a déjà la prop "CodeListReference"
proposition de reformulation :
Property defined on variables associated with a choice question, whose choices are dynamically defined by the responses of another question.
The value of the 'VariableRerference' property is the identifier of the variable defining these dynamic choices.
Therefore, it must only be used when the 'choiceType' in the associated question is 'VARIABLE'.
| <groupId>fr.insee.pogues</groupId> | ||
| <artifactId>pogues-model</artifactId> | ||
| <version>1.15.1</version> | ||
| <version>1.16.0</version> |
There was a problem hiding this comment.
Un détail : si c'est un fix c'est le patch qu'il faut monter
Sinon changer en feat:
|
nsenave
changed the title
nsenave
left a comment
nsenave
left a comment
There was a problem hiding this comment.
Sous réserve que la modélisation est conforme à ce qu'on veut dans Pogues (mais j'imagine que oui)
Bien joué pour les tests @RemiVerriez ! 👍
3 participants
This property is required for the "variable choices in unique choice question" feature, it was missing