HomePhabricator
Diffusion Comm 795467b41fca

[lib] Use channel instead of channel/search Neynar endpoint

Description

[lib] Use channel instead of channel/search Neynar endpoint

Summary:
We're exceeding our compute unit allotment with Neynar: ENG-9930.

It's mostly the channel/search API. I'm guessing it's more expensive than the channel API, which has fewer capabilities.

Turns out we actually already have NeynarClient.fetchFarcasterChannelInfo, which calls the channel API. But we don't have any callsites to it anymore.

This diff replaces the code for NeynarClient.fetchFarcasterChannelByID with NeynarClient.fetchFarcasterChannelInfo, and subsequently removes the latter.

Depends on D13978

Test Plan: Tested that channel avatars still load on both web and native

Reviewers: varun, will

Reviewed By: will

Subscribers: tomek

Differential Revision: https://phab.comm.dev/D13979

Details

Provenance
ashoatAuthored on Sun, Nov 17, 3:52 PM
Reviewer
will
Differential Revision
D13979: [lib] Use channel instead of channel/search Neynar endpoint
Parents
rCOMMac8930d796bb: [lib] Fix bug in getCachedFarcasterChannelEntryForChannelID
Branches
Unknown
Tags
Unknown