Paths

Table of Contentst

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

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

Details

Reviewers

varun
will

Commits

rCOMM4d11d51c931e: [lib] Don't query Neynar for Farcaster channel avatars if unused

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

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

Closed by commit rCOMM4d11d51c931e: [lib] Don't query Neynar for Farcaster channel avatars if unused (authored by ashoat). · Explain WhyFri, Nov 22, 6:13 AM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM4d11d51c931e: [lib] Don't query Neynar for Farcaster channel avatars if unused.

Revision Contents
Changeset List

Path

Size

lib/

components/

farcaster-channel-prefetch-handler.react.js

17 lines

shared/

avatar-utils.js

6 lines

Diff 45907

View Options

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