[keyserver] introduce sendReactionMessageRequestInputValidator to message responders
AbandonedPublic
Actions

Authored by ginsu on Nov 28 2022, 4:27 PM.

Details

Reviewers

atul
rohan
tomek
ashoat

Summary

introduce sendReactionMessageRequestInputValidator to message responders. This will be used to validate the input in reactionMessageCreationResponder (which will be introduced in the subsequent diff)

Linear Task: ENG-2250

Test Plan

This will be tested in subsequent diffs. Also ran web/mobile app locally and nothing crashes/any regressions

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ginsu created this revision.Nov 28 2022, 4:27 PM

Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptNov 28 2022, 4:27 PM

ginsu edited the summary of this revision. (Show Details)Nov 28 2022, 4:28 PM

ginsu edited the summary of this revision. (Show Details)

ginsu edited the summary of this revision. (Show Details)Nov 28 2022, 4:33 PM

ginsu added reviewers: atul, rohan, tomek.

Harbormaster completed remote builds in B13811: Diff 18903.Nov 28 2022, 4:38 PM

ginsu requested review of this revision.Nov 28 2022, 4:38 PM

ginsu added a child revision: D5749: [keyserver] introduced reactionMessageCreationResponder function to message responders.Nov 28 2022, 4:38 PM

What's the Flow type that this corresponds to? I think this would be easier to review if you combined it with the next diff. The ESLint error should be a hint

This revision now requires changes to proceed.Nov 28 2022, 7:01 PM

abandoning because D5749 handles this now

Revision Contents
Changeset List

Path

Size

keyserver/

src/

responders/

message-responders.js

7 lines

Diff 18903

View Options

keyserver/src/responders/message-responders.js

[keyserver] introduce sendReactionMessageRequestInputValidator to message respondersAbandonedPublicActions