[lib] use Comm user IDs in `useFetchConversation`
ClosedPublic
Actions

Authored by kamil on Sep 3 2025, 9:15 AM.

Details

Reviewers

tomek
bartek
angelika

Commits

rCOMM6c410acc92fb: [lib] use Comm user IDs in `useFetchConversation`

Summary

This fix how users are shown in thread

Depends on D15282

Test Plan

Tested later in the stack

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Sep 3 2025, 9:15 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptSep 3 2025, 9:15 AM

kamil added a child revision: D15284: [lib] create Farcaster thread when all users support the protocol.Sep 3 2025, 9:18 AM

Harbormaster failed remote builds in B35616: Diff 50136!Sep 3 2025, 9:20 AM

kamil published this revision for review.Sep 3 2025, 9:20 AM

bartek accepted this revision.Sep 3 2025, 9:46 AM

bartek added inline comments.

lib/shared/farcaster/farcaster-hooks.js
66–74 ↗	(On Diff #50136)	I'd also dispatch action to update UserStore with potentially new members

This revision is now accepted and ready to land.Sep 3 2025, 9:46 AM

address review

Harbormaster failed remote builds in B35621: Diff 50141!Sep 3 2025, 10:56 AM

This revision was landed with ongoing or failed builds.Sep 3 2025, 11:08 AM

Closed by commit rCOMM6c410acc92fb: [lib] use Comm user IDs in `useFetchConversation`. · Explain Why

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM6c410acc92fb: [lib] use Comm user IDs in `useFetchConversation`.

Revision Contents
Changeset List

Path

Size

lib/

shared/

farcaster/

farcaster-hooks.js

27 lines

Diff 50141

View Options

lib/shared/farcaster/farcaster-hooks.js

[lib] use Comm user IDs in `useFetchConversation`ClosedPublicActions