Page MenuHomePhabricator
Differential D11642

[lib] Split `rawUpdateRelationshipMessageInfoValidator` into `rawUpdate[Traditional/Farcaster]RelationshipMessageInfoValidator`
ClosedPublic
Actions

Authored by atul on Thu, Apr 11, 8:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 22, 11:31 AM
Unknown Object (File)
Sun, Apr 21, 11:15 AM
Unknown Object (File)
Sun, Apr 21, 9:28 AM
Unknown Object (File)
Sat, Apr 20, 6:38 PM
Unknown Object (File)
Thu, Apr 18, 8:59 AM
Unknown Object (File)
Wed, Apr 17, 8:18 AM
Unknown Object (File)
Wed, Apr 17, 8:18 AM
Unknown Object (File)
Wed, Apr 17, 8:17 AM
View All 10 Files
Subscribers
ashoat

Details

Reviewers
ginsu
tomek
varun
will
Commits
rCOMMa1d401813b70: [lib] Split `rawUpdateRelationshipMessageInfoValidator` into `rawUpdate…
Summary

Similar to D11641, but for validators.

Depends on D11641

Test Plan

Will add unit tests to this before landing/after rest of stack is up.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul created this revision.Thu, Apr 11, 8:42 PM
Herald added a subscriber: ashoat. · View Herald TranscriptThu, Apr 11, 8:42 PM
Harbormaster completed remote builds in B28182: Diff 39071.Thu, Apr 11, 8:58 PM
atul requested review of this revision.Thu, Apr 11, 8:58 PM
tomek accepted this revision.Fri, Apr 12, 2:41 AM
tomek added inline comments.
lib/types/messages/update-relationship.js
65 ↗(On Diff #39071)
This revision is now accepted and ready to land.Fri, Apr 12, 2:41 AM
atul added a child revision: D11650: [lib] Split `UpdateRelationshipMessageInfo` into `Update[Traditional/Farcaster]RelationshipMessageInfo`.Fri, Apr 12, 11:47 AM
atul added a child revision: D11651: [lib] Include `[creator/target]FID` in `messageContentForServerDB` when update relationship op is `farcaster_mutual`.Fri, Apr 12, 12:21 PM
atul updated this revision to Diff 39155.Tue, Apr 16, 10:51 AM

fix TUnion import

Harbormaster completed remote builds in B28245: Diff 39155.Tue, Apr 16, 11:08 AM
atul removed a child revision: D11651: [lib] Include `[creator/target]FID` in `messageContentForServerDB` when update relationship op is `farcaster_mutual`.Tue, Apr 16, 12:22 PM
atul updated this revision to Diff 39199.Wed, Apr 17, 12:58 PM

rebase BEFORE addressing feedback

atul updated this revision to Diff 39201.Wed, Apr 17, 1:10 PM

address feedback and land

Harbormaster completed remote builds in B28276: Diff 39199.Wed, Apr 17, 1:14 PM
This revision was landed with ongoing or failed builds.Wed, Apr 17, 1:17 PM
Closed by commit rCOMMa1d401813b70: [lib] Split `rawUpdateRelationshipMessageInfoValidator` into `rawUpdate… (authored by atul). · Explain Why
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMMa1d401813b70: [lib] Split `rawUpdateRelationshipMessageInfoValidator` into `rawUpdate….
Harbormaster completed remote builds in B28278: Diff 39201.Wed, Apr 17, 1:26 PM

Revision Contents
Changeset List

PathSize
lib/
types/
messages/
31 lines

Diff 39202

View Options

lib/types/messages/update-relationship.js

Loading...