Page MenuHomePhabricator
Differential D11864

[lib/native] introduce useUnlinkFID
ClosedPublic
Actions

Authored by ginsu on Thu, May 2, 11:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 23, 2:04 PM
Unknown Object (File)
Thu, May 23, 10:41 AM
Unknown Object (File)
Thu, May 23, 10:01 AM
Unknown Object (File)
Thu, May 23, 8:49 AM
Unknown Object (File)
Thu, May 23, 12:18 AM
Unknown Object (File)
Sun, May 12, 1:42 PM
Unknown Object (File)
Sun, May 12, 10:45 AM
Unknown Object (File)
Sun, May 12, 12:37 AM
View All 16 Files
Subscribers
ashoat
tomek
varun

Details

Reviewers
varun
will
ashoat
Commits
rCOMM3d533a2092bb: [lib/native] introduce useUnlinkFID
Summary

This diff introduces useUnlinkFID which is a hook that returns a callback that calls the unlinkFarcasterAccount identity rpc and then clears the fid from the synced metadata client store

Linear task: https://linear.app/comm/issue/ENG-7412/call-unlink-farcaster-rpc-on-the-client

Depends on D11863

Test Plan

Confirmed that the fid was being cleared from the synced metadata client store + confirmed in meeting with @varun that I was correctly calling and getting the expected behavior of unlinkFarcasterAccount RPC

Confirmed that loading + error states worked as expected

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ginsu created this revision.Thu, May 2, 11:42 PM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptThu, May 2, 11:42 PM
ginsu edited the summary of this revision. (Show Details)Thu, May 2, 11:46 PM
ginsu added reviewers: varun, will.
ginsu added a parent revision: D11863: [lib/native] introduce useLinkFID hook.
ginsu edited the summary of this revision. (Show Details)
Harbormaster returned this revision to the author for changes because remote builds failed.Thu, May 2, 11:53 PM
Harbormaster failed remote builds in B28639: Diff 39757!
ginsu added a comment.Thu, May 2, 11:54 PM

will make sure ci passes before landing

ginsu requested review of this revision.Thu, May 2, 11:54 PM
ginsu planned changes to this revision.Fri, May 3, 12:14 PM

Improve error handling

ashoat requested changes to this revision.Fri, May 3, 12:27 PM
ashoat added inline comments.
native/profile/farcaster-account-settings.react.js
31 ↗(On Diff #39757)

What happens on error? Should we show an Alert to the user to explain that it failed?

ashoat added a comment.Fri, May 3, 12:27 PM

(Ah you planned changes before I requested them, cool)

ginsu updated this revision to Diff 39782.Fri, May 3, 11:57 PM

address comments

Harbormaster completed remote builds in B28656: Diff 39782.Sat, May 4, 12:13 AM
ashoat requested changes to this revision.Sat, May 4, 11:38 AM
ashoat added inline comments.
native/profile/farcaster-account-settings.react.js
33 ↗(On Diff #39782)

How do we handle in-progress state here while we’re waiting for the identity service to be called? Should we keep the spinner spinning?

This revision now requires changes to proceed.Sat, May 4, 11:38 AM
ginsu updated this revision to Diff 40141.Mon, May 13, 10:18 PM

address comments

ginsu updated this revision to Diff 40142.Mon, May 13, 10:21 PM

update

ginsu updated this revision to Diff 40143.Mon, May 13, 10:24 PM

forgot to hit save

ginsu edited the test plan for this revision. (Show Details)Mon, May 13, 10:26 PM
Harbormaster completed remote builds in B28882: Diff 40141.Mon, May 13, 10:34 PM
Harbormaster completed remote builds in B28883: Diff 40142.Mon, May 13, 10:37 PM
Harbormaster completed remote builds in B28884: Diff 40143.Mon, May 13, 10:40 PM
ashoat accepted this revision.Tue, May 14, 9:30 AM

Thanks! Please always keep in mind progress and loading state going forward. As a frontend dev, those things need to be second nature to you, and not something you should be relying on reviewers to catch

This revision is now accepted and ready to land.Tue, May 14, 9:30 AM
ginsu updated this revision to Diff 40242.Wed, May 15, 11:27 AM

rebase before landing

Harbormaster completed remote builds in B28958: Diff 40242.Wed, May 15, 11:43 AM
Closed by commit rCOMM3d533a2092bb: [lib/native] introduce useUnlinkFID (authored by ginsu). · Explain WhyWed, May 15, 11:51 AM
This revision was automatically updated to reflect the committed changes.
ginsu added a commit: rCOMM3d533a2092bb: [lib/native] introduce useUnlinkFID.

Revision Contents
Changeset List

PathSize
lib/
utils/
28 lines
native/
profile/
47 lines

Diff 40248

View Options

lib/utils/farcaster-utils.js

Loading...
View Options

native/profile/farcaster-account-settings.react.js

Loading...