fix(qna): reverse qna-questions for consistency

botpress · Sep 28, 2018 · 0043fee · 0043fee
1 parent ed74212
commit 0043fee
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/packages/functionals/botpress-qna/src/providers/qnaMaker.js b/packages/functionals/botpress-qna/src/providers/qnaMaker.js
@@ -101,7 +101,11 @@ export default class Storage {
     const qnas = _.isArray(qna) ? qna : [qna]
     const { data: { operationId } } = await this.patchKb({
       add: {
-        qnaList: qnas.map(qna => ({ ..._.pick(qna, ['answer', 'questions']), metadata: prepareMeta(qna) }))
+        qnaList: qnas.map(qna => ({
+          answer: qna.answer,
+          questions: qna.questions.reverse(), // To be able to prepend questions
+          metadata: prepareMeta(qna)
+        }))
       }
     })
 
@@ -114,7 +118,9 @@ export default class Storage {
   async fetchQuestions() {
     if (!this.questions) {
       const { data: { qnaDocuments } } = await this.client.get(`/knowledgebases/${this.knowledgebase.id}/test/qna/`)
-      this.questions = qnaDocuments
+
+      // Showing latest items first
+      this.questions = qnaDocuments.reverse().map(doc => ({ ...doc, questions: doc.questions.reverse() }))
     }
 
     return this.questions
@@ -135,7 +141,7 @@ export default class Storage {
   async all({ limit, offset } = {}) {
     let questions = await this.fetchQuestions()
     if (typeof limit !== 'undefined' && typeof offset !== 'undefined') {
-      questions = questions.reverse().slice(offset, offset + limit)
+      questions = questions.slice(offset, offset + limit)
     }
 
     return questions.map(qna => ({ id: qna.id, data: qnaItemData(qna) }))