Page MenuHomePhabricator

[lib] modify FC cache to store username and pfp url
ClosedPublic

Authored by varun on Aug 15 2024, 10:52 PM.
Tags
None
Referenced Files
F3197263: D13098.diff
Sat, Nov 9, 7:53 AM
F3188634: D13098.id43589.diff
Fri, Nov 8, 7:02 PM
F3188633: D13098.id43419.diff
Fri, Nov 8, 7:02 PM
F3188602: D13098.diff
Fri, Nov 8, 7:01 PM
Unknown Object (File)
Fri, Nov 8, 2:29 AM
Unknown Object (File)
Wed, Oct 30, 11:24 PM
Unknown Object (File)
Sun, Oct 20, 8:00 PM
Unknown Object (File)
Sun, Oct 20, 4:09 AM
Subscribers

Details

Summary

Depends on D13097

Test Plan

tested later in stack by successfully setting avatar from farcaster

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

varun held this revision as a draft.
varun published this revision for review.Aug 19 2024, 7:18 AM
varun added inline comments.
lib/utils/fc-cache.js
144

wasn't sure about this line

varun added reviewers: ashoat, tomek.
ashoat added inline comments.
lib/utils/fc-cache.js
144

JavaScript is weird... this condition will let null through because typeof null === 'object'

To be fully consistent with the prior implementation, you should add || farcasterUser === null here (although not clear how important that is)

This revision is now accepted and ready to land.Aug 20 2024, 12:33 PM
lib/utils/fc-cache.js
144

Actually I'd just add || !farcasterUser rather than || farcasterUser === null

lib/utils/fc-cache.js
144

my change addressing this feedback wasn't picked up for some reason so i created D13139