[lib] Fetch all the messages after connection and only a portion after opening the app
ClosedPublic
Actions

Authored by tomek on Aug 27 2025, 1:44 PM.

Details

Reviewers

bartek
kamil

Commits

rCOMM599772393598: [lib] Fetch all the messages after connection and only a portion after opening…

Summary

Fetch messages in batches until a limit is reached

Depends on D15250

https://linear.app/comm/issue/ENG-11172/fetch-all-the-farcaster-messages-after-connection

Test Plan

Connect and check if all the messages are fetched. Close and open the app and check if only 20 messages are fetched.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.Aug 27 2025, 1:44 PM

Herald added a subscriber: ashoat. · View Herald TranscriptAug 27 2025, 1:44 PM

tomek published this revision for review.Aug 27 2025, 1:44 PM

Harbormaster failed remote builds in B35554: Diff 50035!Aug 27 2025, 1:45 PM

Fix the function call

Harbormaster failed remote builds in B35555: Diff 50036!Aug 27 2025, 2:08 PM

tomek added a child revision: D15252: [lib] Only insert the first portion of messages to the store.Aug 27 2025, 2:13 PM

kamil accepted this revision.Aug 28 2025, 9:15 AM

kamil added inline comments.

lib/components/farcaster-data-handler.react.js
244–279 ↗	(On Diff #50036)	note: process being killed in the middle is handled in D15257

This revision is now accepted and ready to land.Aug 28 2025, 9:15 AM

This revision was landed with ongoing or failed builds.Aug 28 2025, 10:54 AM

Closed by commit rCOMM599772393598: [lib] Fetch all the messages after connection and only a portion after opening…. · Explain Why

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM599772393598: [lib] Fetch all the messages after connection and only a portion after opening….

Revision Contents
Changeset List

Path

Size

lib/

components/

farcaster-data-handler.react.js

28 lines

shared/

farcaster/

farcaster-api.js

10 lines

farcaster-hooks.js

74 lines

tunnelbroker/

use-peer-to-peer-message-handler.js

2 lines

native/

profile/

profile-screen.react.js

4 lines

web/

settings/

account-settings.react.js

5 lines

Diff 50058

View Options

lib/components/farcaster-data-handler.react.js

View Options

lib/shared/farcaster/farcaster-api.js

View Options

lib/shared/farcaster/farcaster-hooks.js

View Options

lib/tunnelbroker/use-peer-to-peer-message-handler.js

View Options

native/profile/profile-screen.react.js

View Options

web/settings/account-settings.react.js

[lib] Fetch all the messages after connection and only a portion after opening the appClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 50058

lib/components/farcaster-data-handler.react.js

lib/shared/farcaster/farcaster-api.js

lib/shared/farcaster/farcaster-hooks.js

lib/tunnelbroker/use-peer-to-peer-message-handler.js

native/profile/profile-screen.react.js

web/settings/account-settings.react.js

[lib] Fetch all the messages after connection and only a portion after opening the app
ClosedPublic
Actions

Revision Contents
Changeset List