[lib][native][web] Introduce useUpdateRelationships
ClosedPublic
Actions

Authored by ashoat on Mon, Sep 23, 10:40 PM.

Details

Reviewers

tomek

Commits

rCOMMbf4c664c5951: [lib][native][web] Introduce useUpdateRelationships

Summary

This diff updates the updateRelationships action function to use the new update_relationships2 API, and introduces useUpdateRelationships, a new hook that abstracts some complexity and in following diffs will handle forking between thick/thread logic.

Depends on D13440

Test Plan

I tested this new endpoint as part of my testing of ENG-9269

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ashoat created this revision.Mon, Sep 23, 10:40 PM

ashoat added a child revision: D13443: [lib] Add support to useUpdateRelationships for sending relationship robotext to thick threads.Mon, Sep 23, 10:42 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Mon, Sep 23, 10:54 PM

Harbormaster failed remote builds in B31798: Diff 44464!

Rebase

Harbormaster completed remote builds in B31809: Diff 44475.Tue, Sep 24, 12:08 AM

ashoat requested review of this revision.Tue, Sep 24, 12:08 AM

tomek accepted this revision.Tue, Sep 24, 3:31 AM

tomek added inline comments.

lib/components/farcaster-data-handler.react.js
47 ↗	(On Diff #44475)	Does the new endpoint correctly support `farcaster` action without `userIDsToFID` map?

This revision is now accepted and ready to land.Tue, Sep 24, 3:31 AM

ashoat added inline comments.Tue, Sep 24, 10:13 AM

lib/components/farcaster-data-handler.react.js
47 ↗	(On Diff #44475)	D13436 updates our code to ignore this input from the user. Instead of trusting the user's claims about FIDs, we will instead query the identity service