[lib] Don't query Neynar for Farcaster channel avatars if unused
AcceptedPublic
Actions

Authored by ashoat on Wed, Nov 20, 6:48 AM.

Details

Reviewers

varun
will

Summary

This eliminates most fetches, since we mostly aren't using Farcaster channel avatars yet.

Depends on D13982

Test Plan

In combination with other diffs, tested that channel avatars still load on both web and native
Tested on both native and web by logging each NeynarClient query and seeing which ones get triggered

Diff Detail

Repository

rCOMM Comm

Branch

ashoat/diff

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.Wed, Nov 20, 6:48 AM

Herald added a subscriber: tomek. · View Herald TranscriptWed, Nov 20, 6:48 AM

Harbormaster returned this revision to the author for changes because remote builds failed.Wed, Nov 20, 7:06 AM

Harbormaster failed remote builds in B32733: Diff 45907!

Rebase

ashoat published this revision for review.Wed, Nov 20, 7:16 AM

Fix rebase mistake

Harbormaster completed remote builds in B32736: Diff 45910.Wed, Nov 20, 8:25 AM

Harbormaster completed remote builds in B32737: Diff 45911.Wed, Nov 20, 8:33 AM

will accepted this revision.Wed, Nov 20, 2:32 PM

This revision is now accepted and ready to land.Wed, Nov 20, 2:32 PM

Revision Contents
Changeset List

Path

Size

lib/

components/

farcaster-channel-prefetch-handler.react.js

17 lines

shared/

avatar-utils.js

6 lines

Diff 45910

View Options

lib/components/farcaster-channel-prefetch-handler.react.js