Page MenuHomePhabricator
Differential D11452

[lib] Partially update `UpdateRelationshipMessageSpec` to support updated `UpdateRelationship` message type
ClosedPublic
Actions

Authored by atul on Mar 28 2024, 10:44 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 16, 3:46 AM
Unknown Object (File)
Mon, Apr 15, 6:27 PM
Unknown Object (File)
Wed, Apr 10, 3:05 PM
Unknown Object (File)
Wed, Apr 10, 1:16 PM
Unknown Object (File)
Sat, Apr 6, 11:18 AM
Unknown Object (File)
Fri, Apr 5, 1:48 PM
Unknown Object (File)
Wed, Apr 3, 10:13 AM
Unknown Object (File)
Tue, Apr 2, 9:37 PM
View All 12 Files
Subscribers
ashoat

Details

Reviewers
varun
tomek
ginsu
will
Commits
rCOMM99c020412fb6: [lib] Partially update `UpdateRelationshipMessageSpec` to support updated…
Summary

Update UpdateRelationshipMessageSpec type and make necessary changes to resolve flow issues.

Will update implementation of remaining functions to support UpdateRelationship in followup diffs.

Based on following from hackathon branch: https://github.com/CommE2E/comm/commit/c8ed27f5c5a4b23bd181d562741959140a9b473c

Depends on D11443

Test Plan

flow + close reading + logged return values of each of the functions during the hackathon

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
lib/
shared/
messages/
75 lines

Diff 38804

View Options

lib/shared/messages/update-relationship-message-spec.js

Loading...