Page MenuHomePhabricator
Differential D11606

[lib] Update `FarcasterDataHandler` to construct `FarcasterRelationshipRequest` with `userIDsToFID`
ClosedPublic
Actions

Authored by atul on Tue, Apr 9, 8:49 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 26, 8:47 PM
Unknown Object (File)
Fri, Apr 26, 6:04 PM
Unknown Object (File)
Fri, Apr 26, 3:40 PM
Unknown Object (File)
Mon, Apr 22, 7:45 AM
Unknown Object (File)
Mon, Apr 22, 6:56 AM
Unknown Object (File)
Mon, Apr 22, 2:05 AM
Unknown Object (File)
Sun, Apr 21, 6:53 AM
Unknown Object (File)
Thu, Apr 18, 6:26 PM
View All 16 Files
Subscribers
ashoat

Details

Reviewers
ginsu
varun
will
tomek
Commits
rCOMM6e98c6bfd206: [lib] Update `FarcasterDataHandler` to construct `FarcasterRelationshipRequest`…
Summary

We're updating FarcasterDataHandler to construct userIDsToFID map which will be passed to the keyserver. We include userIDs and fids for both the "target users" as well as current user.

NOTE: The request constructed on the client will currently fail. We need to update both validators and responder on keyserver to support this new format.

Depends on D11605

Test Plan

Add log statements to userIDsToFID, userIDsToFIDIncludingCurrentUser, and createThreadsAndRobotextForFarcasterMutuals and ensure that data is structured as expected.

(made some tweaks to the code here, so will go through again before landing)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul created this revision.Tue, Apr 9, 8:49 PM
Herald added a subscriber: ashoat. · View Herald TranscriptTue, Apr 9, 8:49 PM
Harbormaster completed remote builds in B28107: Diff 38963.Tue, Apr 9, 9:06 PM
atul requested review of this revision.Tue, Apr 9, 9:06 PM
atul added a child revision: D11607: [keyserver] Update `createPersonalThreads` to support both `[Traditional/Farcaster]RelationshipRequest`.Tue, Apr 9, 9:17 PM
tomek accepted this revision.Wed, Apr 10, 6:43 AM
NOTE: The request constructed on the client will currently fail. We need to update both validators and responder on keyserver to support this new format.

A safer ordering might be to first introduce the support for a new format on the keyserver, gated behind code version, and then modify the format on the client.

lib/components/farcaster-data-handler.react.js
30 ↗(On Diff #38963)
92–98 ↗(On Diff #38963)

Up to you, but Object.fromEntries might be simpler.

ashoat added inline comments.Wed, Apr 10, 8:47 AM
lib/components/farcaster-data-handler.react.js
24 ↗(On Diff #38963)

I think we can now remove this line, and instead define loggedIn based on !!currentUserID

atul updated this revision to Diff 39001.Wed, Apr 10, 12:06 PM
atul marked an inline comment as done.

address @tomek's feedback

atul added inline comments.Wed, Apr 10, 12:10 PM
lib/components/farcaster-data-handler.react.js
24 ↗(On Diff #38963)

Will update with this change

92–98 ↗(On Diff #38963)

That's a lot cleaner and more concise, thanks for the suggestion!

atul updated this revision to Diff 39004.Wed, Apr 10, 12:14 PM

address @ashoat's feedback

ginsu accepted this revision.Wed, Apr 10, 12:25 PM
This revision is now accepted and ready to land.Wed, Apr 10, 12:25 PM
Harbormaster completed remote builds in B28135: Diff 39001.Wed, Apr 10, 12:25 PM
Harbormaster completed remote builds in B28136: Diff 39004.Wed, Apr 10, 12:33 PM
atul mentioned this in D11669: [native] Redux migration to clear `UNSUPPORTED<UPDATE_RELATIONSHIP>` messages.Tue, Apr 16, 12:23 PM
atul mentioned this in D11609: [lib] Narrow `TraditionalRelationshipRequest` to exclude 'farcaster' action.Wed, Apr 17, 12:24 PM
atul updated this revision to Diff 39248.Thu, Apr 18, 1:20 PM

rebase and land

This revision was landed with ongoing or failed builds.Thu, Apr 18, 1:26 PM
Closed by commit rCOMM6e98c6bfd206: [lib] Update `FarcasterDataHandler` to construct `FarcasterRelationshipRequest`… (authored by atul). · Explain Why
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMM6e98c6bfd206: [lib] Update `FarcasterDataHandler` to construct `FarcasterRelationshipRequest`….
Harbormaster completed remote builds in B28307: Diff 39248.Thu, Apr 18, 1:42 PM
atul mentioned this in rCOMMbf26a3ce015a: [native] Redux migration to clear `UNSUPPORTED<UPDATE_RELATIONSHIP>` messages.Fri, Apr 19, 11:37 AM

Revision Contents
Changeset List

PathSize
lib/
components/
39 lines

Diff 39254

View Options

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

Loading...