Page MenuHomePhabricator

[keyserver] Show the most recent thread when opening the web app without providing a thread ID
ClosedPublic

Authored by tomek on Thu, Oct 3, 7:52 AM.
Tags
None
Referenced Files
F2929144: D13590.diff
Wed, Oct 9, 12:27 PM
F2927083: D13590.diff
Tue, Oct 8, 8:21 PM
Unknown Object (File)
Sat, Oct 5, 5:15 AM
Unknown Object (File)
Fri, Oct 4, 9:29 PM
Unknown Object (File)
Fri, Oct 4, 8:52 AM
Unknown Object (File)
Thu, Oct 3, 7:53 PM
Unknown Object (File)
Thu, Oct 3, 5:36 PM
Unknown Object (File)
Thu, Oct 3, 2:12 PM
Subscribers

Details

Summary

The most recent thread can be thick and the keyserver doesn't know about it. In this case we can rely on the app to compute the activeChatThreadID instead of the keyserver. In validateStateAndQueueOpsProcessing we're handling an empty state.navInfo.activeChatThreadID by calling mostRecentlyReadThreadSelector. This means that we can simply stop handling empty activeChatThreadID on the keyserver side.

https://linear.app/comm/issue/ENG-9126/opening-a-link-to-the-chat-doesnt-open-the-most-recent-thick-thread

Test Plan

Made sure that the most recent thread is thick. Opened /webapp and /webapp/chat and verified that the most recent thread is shown.
Updated the state so that a thin thread is the most recent and performed the same test.
Made sure that opening a link to a thin and to a thick thread still opens it even if it isn't the most recent.

Diff Detail

Repository
rCOMM Comm
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage