From 0043fee5ee023ba70857ef7e7ff76689644f3a8a Mon Sep 17 00:00:00 2001 From: Oleksiy Pletnov Date: Fri, 28 Sep 2018 00:23:13 +0300 Subject: [PATCH] fix(qna): reverse qna-questions for consistency --- .../botpress-qna/src/providers/qnaMaker.js | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/packages/functionals/botpress-qna/src/providers/qnaMaker.js b/packages/functionals/botpress-qna/src/providers/qnaMaker.js index 0c2896c7034..da75d71b353 100644 --- 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) }))