Page MenuHomePhabricator
Differential D11605

[lib] Re-introduce `RelationshipRequest` and replace `TraditionalRelationshipRequest` where simple
ClosedPublic
Actions

Authored by atul on Tue, Apr 9, 8:06 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 26, 7:40 AM
Unknown Object (File)
Mon, Apr 22, 6:56 PM
Unknown Object (File)
Sun, Apr 21, 8:35 PM
Unknown Object (File)
Sun, Apr 21, 5:14 PM
Unknown Object (File)
Sun, Apr 21, 9:36 AM
Unknown Object (File)
Thu, Apr 18, 3:19 PM
Unknown Object (File)
Thu, Apr 18, 1:34 PM
Unknown Object (File)
Thu, Apr 18, 12:33 PM
View All 16 Files
Subscribers
ashoat

Details

Reviewers
varun
ginsu
will
tomek
Commits
rCOMMeef5dc5615bc: [lib] Re-introduce `RelationshipRequest` and replace…
Summary

Re-introduce RelationshipRequest as union of TraditionalRelationshipRequest|FarcasterRelationship and replace TraditionalRelationshipRequest where simple.

Next steps are to

  1. Narrow TraditionalRelationshipRequest so it excludes farcaster action
  2. Update FarcasterRelationshipRequest callsite
  3. Make necessary changes on keyserver to accomodate

Depends on D11603

Test Plan

flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul created this revision.Tue, Apr 9, 8:06 PM
Herald added a subscriber: ashoat. · View Herald TranscriptTue, Apr 9, 8:06 PM
atul updated this revision to Diff 38962.Tue, Apr 9, 8:07 PM

fix

Harbormaster failed remote builds in B28105: Diff 38961!Tue, Apr 9, 8:18 PM
Harbormaster completed remote builds in B28106: Diff 38962.Tue, Apr 9, 8:23 PM
atul requested review of this revision.Tue, Apr 9, 8:23 PM
atul added a child revision: D11606: [lib] Update `FarcasterDataHandler` to construct `FarcasterRelationshipRequest` with `userIDsToFID`.Tue, Apr 9, 8:49 PM
ginsu accepted this revision.Tue, Apr 9, 10:07 PM
This revision is now accepted and ready to land.Tue, Apr 9, 10:07 PM
atul updated this revision to Diff 38999.Wed, Apr 10, 11:53 AM

land

This revision was landed with ongoing or failed builds.Wed, Apr 10, 11:54 AM
Closed by commit rCOMMeef5dc5615bc: [lib] Re-introduce `RelationshipRequest` and replace… (authored by atul). · Explain Why
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMMeef5dc5615bc: [lib] Re-introduce `RelationshipRequest` and replace….
Harbormaster completed remote builds in B28134: Diff 38999.Wed, Apr 10, 12:10 PM

Revision Contents
Changeset List

PathSize
lib/
actions/
6 lines
types/
4 lines
native/
profile/
4 lines

Diff 39000

View Options

lib/actions/relationship-actions.js

Loading...
View Options

lib/types/relationship-types.js

Loading...
View Options

native/profile/relationship-list-item.react.js

Loading...