Chatbot 630 documentary resources display && Chatbot 624 / 671 / 673 platform redesign #251
Conversation
romaricspklee
changed the title
romaricspklee
changed the title
…ceDisplay params]
romaricspklee
force-pushed
the
chatbot-630-documentary-resources-display
branch
from
880fe8d
to
7602916
Compare
romaricspklee
changed the title
.../src/main/java/io/vertigo/chatbot/designer/builder/services/DocumentaryResourceServices.java
Show resolved
Hide resolved
vertigo-bot-commons/src/main/resources/io/vertigo/chatbot/commons/mda/webservices.ksp
Show resolved
Hide resolved
vertigo-bot-commons/src/main/resources/io/vertigo/chatbot/commons/mda/webservices.ksp
Outdated
Show resolved
Hide resolved
| } | ||
| } | ||
| return documentaryResourceExport; | ||
| }).collect(VCollectors.toDtList(DocumentaryResourceExport.class))); |
There was a problem hiding this comment.
Il me semblerait plus efficace, plutôt que de peupler la liste des documentaryResourceExport en Java, en bouclant sur chaque ressource et en les chargeant une à une, de récupérer cette liste directement à l'aide d'une requête en base.
Le seul point bloquant pour cette requête était la récupération du contenu de l'éventuel fichier associé, récupération supprimée dans le commit 9f31d23
...c/main/java/io/vertigo/chatbot/designer/builder/services/topic/export/BotExportServices.java
Show resolved
Hide resolved
...go-bot-executor/src/main/java/io/vertigo/chatbot/executor/manager/ExecutorConfigManager.java
Outdated
Show resolved
Hide resolved
| } | ||
|
|
||
| @GET("/getDocumentaryResourceFile") | ||
| public VFile getDocumentaryResourceFile(@QueryParam("label") final String label) { |
There was a problem hiding this comment.
Le libellé d'un document (utilisé ici comme paramètre) n'est pas unique, et ne permet donc pas d'identifier correctement le fichier à récupérer.
There was a problem hiding this comment.
Oui, je n'ai pas le détail de pourquoi c'est fait comme ça, mais il faudrait plutôt se baser sur un ID. A corriger
There was a problem hiding this comment.
Corrigé pour les ressources documentaires, mais le problème subsiste pour les pièces jointes, à cause d'un sujet fonctionnel (cf Jira CHATBOT-692)
cf commit ace3f6b , PR #255
No description provided.