Page MenuHomePhabricator

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

Authored by ashoat on Nov 20 2024, 6:44 AM.
Tags
None
Referenced Files
F3508885: D13979.id.diff
Sat, Dec 21, 12:15 AM
F3508511: D13979.id45966.diff
Fri, Dec 20, 10:35 PM
Unknown Object (File)
Thu, Dec 19, 10:30 AM
Unknown Object (File)
Sat, Dec 14, 4:32 PM
Unknown Object (File)
Sat, Dec 14, 1:20 PM
Unknown Object (File)
Fri, Dec 13, 2:15 AM
Unknown Object (File)
Mon, Dec 9, 4:40 AM
Unknown Object (File)
Thu, Dec 5, 12:51 PM
Subscribers

Details

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

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable