Chatbot 632 setting use of context values #253
Conversation
…ceDisplay params]
…d QuestionAnswer]
…e and question/answer]
… delete and on contextValue delete ++ update delete bot method]
| cva.cva_id, | ||
| cpv.cpv_id, | ||
| cva.label as cva_label, | ||
| cpv.value as cpv_value |
There was a problem hiding this comment.
À l'affichage, il me semble préférable de présenter à l'utilisateur une valeur "opérateur : valeur" plutôt que seulement la valeur.
Le champ cpv_value récupéré ici est donc pour moi à alimenter en ce sens.
There was a problem hiding this comment.
J'aime bien l'idée, à valider avec notre cher PO :)
| cva.cva_id, | ||
| cpv.cpv_id, | ||
| cva.label as cva_label, | ||
| cpv.value as cpv_value |
There was a problem hiding this comment.
Même remarque que pour les ressources documentaires : il me semble préférable de récupérer une valeur "opérateur : valeur" pour le cpv_value.
There was a problem hiding this comment.
Idem que précedemment, à valider avec le PO
| <vu:include-data object="documentaryResourceTypeList" field="dreTypeCd" /> | ||
| <vu:include-data object="documentaryResource" field="*" modifiable="true"/> | ||
| <vu:include-data object="documentaryResourceContextIhmList" field="*" modifiable="true"/> | ||
| <vu:include-data object="documentaryResourceTypeList" field="*" /> |
There was a problem hiding this comment.
Suite à cette modifications, les 2 include-data suivants devraient pouvoir être supprimés.
There was a problem hiding this comment.
Oui, effectivement
| v-model="vueData.newDocumentaryResourceContext.cpvId" | ||
| :options="[{ cpvId: null, value: '' }, ...vueData.filteredContextPossibleValueList]" | ||
| option-value="cpvId" | ||
| option-label="value" |
There was a problem hiding this comment.
Pour la valeur de contexte à sélectionner, il vaudrait mieux afficher à l'utilisateur "opérateur : valeur" plutôt que la valeur seule.
There was a problem hiding this comment.
Idem, à valider avec le PO
| fldDocumentaryResourceContextIhm$cvaId = Context value id | ||
| fldDocumentaryResourceContextIhm$cpvId = Context possible value id | ||
| fldDocumentaryResourceContextIhm$cvaLabel = Context | ||
| fldDocumentaryResourceContextIhm$cpvValue = Value |
There was a problem hiding this comment.
Ajouter également ces clés de ressources dans les fichiers d'internationalisation.
| v-model="vueData.newQuestionAnswerContext.cpvId" | ||
| :options="[{ cpvId: null, value: '' }, ...vueData.filteredContextPossibleValueList]" | ||
| option-value="cpvId" | ||
| option-label="value" |
There was a problem hiding this comment.
De même que pour les ressources documentaires, il vaudrait pour moi mieux afficher à l'utilisateur "opérateur : valeur".
There was a problem hiding this comment.
A valider avec le PO
| fldQuestionAnswerContextIhm$cvaId = Context value id | ||
| fldQuestionAnswerContextIhm$cpvId = Context possible value id | ||
| fldQuestionAnswerContextIhm$cvaLabel = Context | ||
| fldQuestionAnswerContextIhm$cpvValue = Value |
There was a problem hiding this comment.
Ajouter également ces clés de ressources dans les fichiers d'internationalisation.
| chatbotProfilServices.deleteAllProfilByBot(bot); | ||
| chatbotCustomConfigServices.deleteChatbotCustomConfig(bot); | ||
| contextValueServices.deleteAllByBotId(bot.getBotId()); | ||
| contextPossibleValueServices.deleteAllContextPossibleValueByBot(bot); |
There was a problem hiding this comment.
Il me semblerait préférable que ce soit le service de suppression des contextValue (contextValueServices.deleteAllByBotId(bot)) qui s'occupe de supprimer les contextPossibleValue associées.
There was a problem hiding this comment.
Oui, plus logique, à corriger
No description provided.