feat: qcu options based on loop response#182
Conversation
| </xs:enumeration> | ||
| <xs:enumeration value="SUGGESTER"> | ||
| <xs:annotation> | ||
| <xs:documentation>For a large list of codes, such as a nomenclature, this component is used to suggest the sublist of codes |
There was a problem hiding this comment.
Can you add a line saying that this parameter will (hopefully) soon to be deprecated ? (replaced by choiceType), it should stay here for now only for retro-compatibility
| </xs:documentation> | ||
| </xs:annotation> | ||
| </xs:enumeration> | ||
| <xs:enumeration value="VARIABLE_RESPONSES"> |
There was a problem hiding this comment.
Small typo here, it's VARIABLE instead of VARIABLE_RESPONSES
nsenave
left a comment
nsenave
left a comment
There was a problem hiding this comment.
👍
quelques remarques ; pas forcément besoin de retoucher le code
| <xs:complexType name="ResponseType"> | ||
| <xs:sequence> | ||
| <xs:element name="CodeListReference" type="xs:token" minOccurs="0"/> | ||
| <xs:element name="VariableReference" type="xs:token" minOccurs="0"> |
There was a problem hiding this comment.
à vérifier que le front de pogues mets une majuscule au début du nom de la prop (je trouve pas de PR sur le dépôt de pogues)
|
|
||
| assertTrue(json.contains("\"choiceType\":\"VARIABLE\"")); | ||
| assertTrue(json.contains("\"OptionFilter\":\"nvl($AGE$, 0) > 18\"")); | ||
| assertTrue(json.contains("\"VariableReference\":\"id-loop-variable\"")); |
There was a problem hiding this comment.
privilégier des tests avec JSONAssert en général pour du contenu json mais ok
There was a problem hiding this comment.
👍 pour le formatting plus cohérent, mais à privilégier dans une PR / commit sur branche de dev style: ... pour rendre les changements facile à suivre
| "id-loop-variable", | ||
| question.getResponse().getFirst().getVariableReference() | ||
| ); | ||
| } |
There was a problem hiding this comment.
on pourrait grouper quelques-unes de ces méthodes de test mais ok
| </xs:annotation> | ||
| </xs:element> | ||
| <xs:element name="OptionFilter" type="xs:string" minOccurs="0"> | ||
| <xs:annotation> |
There was a problem hiding this comment.
pareil que pour "VariableReference", à vérifier que la prop commence aussi par une majuscule côté front
NB : historiquement dans Pogues-Model, la plupart des props avaient cette convention de nommage, mais c'est pas dans les standards json (notamment) donc on essaye de respecter le format somePropName pour les nouvelles props
There was a problem hiding this comment.
Pas encore décidé ni implémenté pour l'instant, je ne suis même plus sure si c'est dans le cadre de la carte
UPDATE: non
3 participants
Schema changes
Introduced a new enum ChoiceTypeEnum with values:
Added a new attribute choiceType to QuestionType (default: CODE_LIST)
Added a new element OptionFilter to QuestionType
Added a new element VariableReference to ResponseType
Tests
Added JSON deserialization tests covering:
Added JSON serialization test to ensure correct output generation.