Page MenuHomePhabricator

[lib] add post cast to neynar client
AcceptedPublic

Authored by will on Wed, Oct 23, 11:43 PM.
Tags
None
Referenced Files
F3060628: D13784.id45367.diff
Thu, Oct 24, 8:23 AM
F3059771: D13784.id45367.diff
Thu, Oct 24, 4:36 AM
F3059761: D13784.id.diff
Thu, Oct 24, 4:35 AM
F3059757: D13784.diff
Thu, Oct 24, 4:32 AM
F3059570: D13784.id45367.diff
Thu, Oct 24, 4:09 AM
F3059541: D13784.diff
Thu, Oct 24, 3:55 AM
F3059368: D13784.id.diff
Thu, Oct 24, 3:33 AM
F3058959: D13784.diff
Thu, Oct 24, 2:39 AM
Subscribers

Details

Reviewers
ashoat
varun
Summary

This implements posting a cast using the neynar client.

Followed https://docs.neynar.com/reference/post-cast almost exactly.

Depends on D13783

Test Plan

Unfortunately, something seems to be wrong with the signer uuid configured in our comm neynar account. Debugging, but will ensure a reply cast is properly posted bofore landing

[NODEM]   code: 'NotFound',
[NODEM]   message: 'Signer not found.',
[NODEM]   property: 'signer_uuid'
[NODEM] }

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

will published this revision for review.Wed, Oct 23, 11:52 PM
will edited the test plan for this revision. (Show Details)
will edited the test plan for this revision. (Show Details)
ashoat added inline comments.
lib/utils/neynar-client.js
384–386

Nit: not sure this variable adds much?

This revision is now accepted and ready to land.Thu, Oct 24, 6:23 AM