From 71fc86b9a6f5e2250f5ca557a9f793c6c1879716 Mon Sep 17 00:00:00 2001 From: Wentao Lyu <35-wentao.lyu@users.noreply.git.stereye.tech> Date: Wed, 15 Mar 2023 02:43:21 +0800 Subject: [PATCH] fix: buildTree should store parent-not-exist message as root. rather than dropping them. --- client/src/components/Messages/index.jsx | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/client/src/components/Messages/index.jsx b/client/src/components/Messages/index.jsx index 5d191e61cf..85c671d691 100644 --- a/client/src/components/Messages/index.jsx +++ b/client/src/components/Messages/index.jsx @@ -34,14 +34,11 @@ const Messages = ({ messages }) => { messages.forEach(message => { messageMap[message.messageId] = {...message, children: []}; - if (message.parentMessageId === "00000000-0000-0000-0000-000000000000") { + const parentMessage = messageMap[message.parentMessageId]; + if (parentMessage) + parentMessage.children.push(messageMap[message.messageId]); + else rootMessages.push(messageMap[message.messageId]); - } else { - const parentMessage = messageMap[message.parentMessageId]; - if (parentMessage) { - parentMessage.children.push(messageMap[message.messageId]); - } - } }); return rootMessages;