[lib] Fetch all the messages when we realize a thread is new
ClosedPublic
Actions

Authored by tomek on Sep 11 2025, 3:13 PM.

Details

Reviewers

kamil
bartek

Commits

rCOMM1e0cd2ff7841: [lib] Fetch all the messages when we realize a thread is new

Summary

If we're fetching a subset of messages and see a new thread, we should instead fetch all the messages.

https://linear.app/comm/issue/ENG-11186/fetch-all-the-messages-when-a-user-is-added-to-a-thread

Test Plan

Tested later in the stack

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.Sep 11 2025, 3:13 PM

Herald added a subscriber: ashoat. · View Herald TranscriptSep 11 2025, 3:13 PM

tomek added a child revision: D15329: [lib] Fetch all the messages when we receive a message from an unknown conversation.Sep 11 2025, 3:14 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Sep 11 2025, 3:29 PM

Harbormaster failed remote builds in B35693: Diff 50252!

Rebase

tomek published this revision for review.Sep 15 2025, 10:33 AM

Harbormaster failed remote builds in B35729: Diff 50306!Sep 15 2025, 10:38 AM

tomek edited the test plan for this revision. (Show Details)Sep 15 2025, 12:29 PM

kamil accepted this revision.Sep 15 2025, 1:43 PM

This revision is now accepted and ready to land.Sep 15 2025, 1:43 PM

This revision was landed with ongoing or failed builds.Sep 18 2025, 1:09 PM

Closed by commit rCOMM1e0cd2ff7841: [lib] Fetch all the messages when we realize a thread is new. · Explain Why

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM1e0cd2ff7841: [lib] Fetch all the messages when we realize a thread is new.

Revision Contents
Changeset List

Path

Size

lib/

shared/

farcaster/

farcaster-hooks.js

18 lines

Diff 50389

View Options

lib/shared/farcaster/farcaster-hooks.js

[lib] Fetch all the messages when we realize a thread is newClosedPublicActions