chore: move msgId to commonProps in SessionMessagesList

pull/3281/head
Audric Ackermann 3 months ago
parent f3cfe57764
commit 3c0f9fe14e
No known key found for this signature in database

@ -49,16 +49,14 @@ export const SessionMessagesList = (props: {
useLayoutEffect(() => { useLayoutEffect(() => {
const newTopMessageId = messagesProps.length const newTopMessageId = messagesProps.length
? messagesProps[messagesProps.length - 1].message.props.messageId ? messagesProps[messagesProps.length - 1].messageId
: undefined; : undefined;
if (oldTopMessageId !== newTopMessageId && oldTopMessageId && newTopMessageId) { if (oldTopMessageId !== newTopMessageId && oldTopMessageId && newTopMessageId) {
props.scrollAfterLoadMore(oldTopMessageId, 'load-more-top'); props.scrollAfterLoadMore(oldTopMessageId, 'load-more-top');
} }
const newBottomMessageId = messagesProps.length const newBottomMessageId = messagesProps.length ? messagesProps[0].messageId : undefined;
? messagesProps[0].message.props.messageId
: undefined;
if (newBottomMessageId !== oldBottomMessageId && oldBottomMessageId && newBottomMessageId) { if (newBottomMessageId !== oldBottomMessageId && oldBottomMessageId && newBottomMessageId) {
props.scrollAfterLoadMore(oldBottomMessageId, 'load-more-bottom'); props.scrollAfterLoadMore(oldBottomMessageId, 'load-more-bottom');
@ -93,7 +91,7 @@ export const SessionMessagesList = (props: {
return ( return (
<IsDetailMessageViewContext.Provider value={false}> <IsDetailMessageViewContext.Provider value={false}>
{messagesProps.map(messageProps => { {messagesProps.map(messageProps => {
const messageId = messageProps.message.props.messageId; const { messageId } = messageProps;
const unreadIndicator = messageProps.showUnreadIndicator ? ( const unreadIndicator = messageProps.showUnreadIndicator ? (
<SessionLastSeenIndicator <SessionLastSeenIndicator
key={'unread-indicator'} key={'unread-indicator'}

@ -140,15 +140,18 @@ export const getSortedMessagesTypesOfSelectedConversation = createSelector(
? messageTimestamp ? messageTimestamp
: undefined; : undefined;
const common = { showUnreadIndicator: isFirstUnread, showDateBreak }; const common = {
const messageIdProps = { messageId: msg.propsForMessage.id }; showUnreadIndicator: isFirstUnread,
showDateBreak,
messageId: msg.propsForMessage.id,
};
if (msg.propsForDataExtractionNotification) { if (msg.propsForDataExtractionNotification) {
return { return {
...common, ...common,
message: { message: {
messageType: 'data-extraction', messageType: 'data-extraction',
props: { ...msg.propsForDataExtractionNotification, ...messageIdProps }, props: { ...msg.propsForDataExtractionNotification },
}, },
}; };
} }
@ -158,7 +161,6 @@ export const getSortedMessagesTypesOfSelectedConversation = createSelector(
...common, ...common,
message: { message: {
messageType: 'message-request-response', messageType: 'message-request-response',
props: messageIdProps,
}, },
}; };
} }
@ -168,7 +170,6 @@ export const getSortedMessagesTypesOfSelectedConversation = createSelector(
...common, ...common,
message: { message: {
messageType: 'group-invitation', messageType: 'group-invitation',
props: messageIdProps,
}, },
}; };
} }
@ -178,7 +179,7 @@ export const getSortedMessagesTypesOfSelectedConversation = createSelector(
...common, ...common,
message: { message: {
messageType: 'group-notification', messageType: 'group-notification',
props: { ...msg.propsForGroupUpdateMessage, ...messageIdProps }, props: { ...msg.propsForGroupUpdateMessage },
}, },
}; };
} }
@ -188,7 +189,7 @@ export const getSortedMessagesTypesOfSelectedConversation = createSelector(
...common, ...common,
message: { message: {
messageType: 'timer-notification', messageType: 'timer-notification',
props: { ...msg.propsForTimerNotification, ...messageIdProps }, props: { ...msg.propsForTimerNotification },
}, },
}; };
} }
@ -200,7 +201,6 @@ export const getSortedMessagesTypesOfSelectedConversation = createSelector(
messageType: 'call-notification', messageType: 'call-notification',
props: { props: {
...msg.propsForCallNotification, ...msg.propsForCallNotification,
...messageIdProps,
}, },
}, },
}; };
@ -213,18 +213,15 @@ export const getSortedMessagesTypesOfSelectedConversation = createSelector(
messageType: 'interaction-notification', messageType: 'interaction-notification',
props: { props: {
...msg.propsForInteractionNotification, ...msg.propsForInteractionNotification,
...messageIdProps,
}, },
}, },
}; };
} }
return { return {
showUnreadIndicator: isFirstUnread, ...common,
showDateBreak,
message: { message: {
messageType: 'regular-message', messageType: 'regular-message',
props: messageIdProps,
}, },
}; };
}); });

Loading…
Cancel
Save