Closed estrattonbailey closed 2 weeks ago
Overall looking really solid!
Implementation seems sound & tests looks solid.
After thinking it through, I think we should reconsider knownFollowers
as a new top level thing on the Hydrator and reflect that in the hydrator, actor hydrator & views class instead of parameterizing the current methods. Lmk what you think/if you have any questions
Adds
knownFollowers
toViewerState
onProfileDetailed
views. This includes:getProfiles
getProfile
getSuggestedFollowsByActor
getSuggestions
Also adds
getKnownFollowers
method that returns the full list of known followers for a givenactor
. This is set up for pagination, but is not actually paginated atm due to dataplane constraints.