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)
Sun, Jan 12, 9:47 AM
Unknown Object (File)
Sat, Jan 11, 11:45 AM
Unknown Object (File)
Thu, Jan 9, 6:48 AM
Unknown Object (File)
Thu, Jan 9, 6:33 AM
Unknown Object (File)
Sun, Jan 5, 4:23 AM
Unknown Object (File)
Jan 3 2025, 8:00 AM
Unknown Object (File)
Dec 27 2024, 1:40 AM
Unknown Object (File)
Dec 27 2024, 1:40 AM
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