Page MenuHomePhabricator

[keyserver] implemented FarcasterChannelTagBlob type for upload payload with validator
ClosedPublic

Authored by will on Oct 22 2024, 9:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 28, 2:51 PM
Unknown Object (File)
Wed, Nov 27, 1:54 PM
Unknown Object (File)
Nov 23 2024, 5:37 AM
Unknown Object (File)
Nov 22 2024, 5:29 PM
Unknown Object (File)
Nov 22 2024, 5:29 PM
Unknown Object (File)
Nov 22 2024, 5:29 PM
Unknown Object (File)
Nov 22 2024, 5:28 PM
Unknown Object (File)
Nov 18 2024, 3:51 PM
Subscribers

Details

Summary

Made a type for the FarcasterChannelTagBlob upload payload with a validator.

Later in the stack, I'll be downloading this blob and will need to validate to ensure proper type checking

Depends on D13773

Test Plan

flow check

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

will requested review of this revision.Oct 22 2024, 9:43 PM
keyserver/src/creators/farcaster-channel-tag-creator.js
149–154 ↗(On Diff #45332)

Unsure if this is the right place to put it. Maybe lib/utils/farcaster-utils.js?

ashoat added inline comments.
keyserver/src/creators/farcaster-channel-tag-creator.js
149–154 ↗(On Diff #45332)

Might be better. Do you think it'll be used outside of keyserver?

This revision is now accepted and ready to land.Oct 23 2024, 5:40 AM
will added inline comments.
keyserver/src/creators/farcaster-channel-tag-creator.js
149–154 ↗(On Diff #45332)

Unsure. I don't think so but this might be a question for @varun or @bartek

keyserver/src/creators/farcaster-channel-tag-creator.js
149–154 ↗(On Diff #45332)

Going to keep this in here for now. Can't think of an example right now where clients would use this