[native] Fix handleCurrentUserFID querying for current user's FID
ClosedPublic
Actions

Authored by ashoat on Sep 12 2024, 2:01 PM.

Details

Reviewers

varun
will

Commits

rCOMM0d12ff46e350: [native] Fix handleCurrentUserFID querying for current user's FID

Summary

This addresses ENG-9230. The cause was a simple error from D12569. Flow failed to catch it because it wasn't sure that currentUserID wasn't the string "identities".

Test Plan

I deployed a release build to my iOS device and confirmed that the Profile -> Farcaster account screen now showed a linked Farcaster account.

Diff Detail

Repository

rCOMM Comm

Branch

ashoat/farcasterdatahandler

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.Sep 12 2024, 2:01 PM

Herald added a subscriber: tomek. · View Herald TranscriptSep 12 2024, 2:01 PM

Harbormaster completed remote builds in B31580: Diff 44113.Sep 12 2024, 2:18 PM

ashoat requested review of this revision.Sep 12 2024, 2:18 PM

varun accepted this revision.Sep 13 2024, 1:51 AM

This revision is now accepted and ready to land.Sep 13 2024, 1:51 AM

Closed by commit rCOMM0d12ff46e350: [native] Fix handleCurrentUserFID querying for current user's FID (authored by ashoat). · Explain WhySep 13 2024, 7:52 AM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM0d12ff46e350: [native] Fix handleCurrentUserFID querying for current user's FID.

Revision Contents
Changeset List

Path

Size

lib/

components/

farcaster-data-handler.react.js

6 lines

Diff 44113

View Options

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

[native] Fix handleCurrentUserFID querying for current user's FIDClosedPublicActions