Paths

Table of Contentst

Differential D13239

[lib] Don't assume that threads exist in thread store when merging new messages
ClosedPublic
Actions

Authored by tomek on Sep 4 2024, 12:56 PM.

Tags

None

Referenced Files

None

Subscribers

Details

Reviewers

kamil
inka

Commits

rCOMM80eb2670dcc7: [lib] Don't assume that threads exist in thread store when merging new messages

Summary

We don't store pending threads in thread store.

https://linear.app/comm/issue/ENG-9140/incorrect-handling-of-startreached-in-message-reducer

Test Plan

Check if creating thin and thick threads is possible.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.Sep 4 2024, 12:56 PM

Herald added a subscriber: ashoat. · View Herald TranscriptSep 4 2024, 12:56 PM

Harbormaster completed remote builds in B31427: Diff 43882.Sep 4 2024, 1:30 PM

tomek requested review of this revision.Sep 4 2024, 1:30 PM

This fixes my issue, thank you!

This revision is now accepted and ready to land.Sep 4 2024, 3:31 PM

Closed by commit rCOMM80eb2670dcc7: [lib] Don't assume that threads exist in thread store when merging new messages. · Explain WhySep 4 2024, 3:53 PM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM80eb2670dcc7: [lib] Don't assume that threads exist in thread store when merging new messages.

ashoat added inline comments.Sep 9 2024, 12:02 PM

lib/reducers/message-reducer.js
628	This code might have a bug for thick sidebars. Looking at `parsePendingThreadID`, it seems like for pending sidebars, we assume `threadTypes.SIDEBAR`.

ashoat added a subscriber: marcin.Sep 9 2024, 3:27 PM

ashoat added inline comments.

lib/reducers/message-reducer.js
628	Actually, looks like @marcin already addressed this

tomek added inline comments.Sep 10 2024, 7:36 AM

lib/reducers/message-reducer.js
628	This was fixed in https://linear.app/comm/issue/ENG-9142/pending-thread-ids-dont-support-thick-sidebars.

Revision Contents
Changeset List

Path

Size

lib/

reducers/

message-reducer.js

33 lines

Diff 43884

lib/reducers/message-reducer.js

Loading...