[lib] Update union types in `message-types` to include `UpdateRelationship` types
ClosedPublic
Actions

Authored by atul on Mar 28 2024, 3:00 PM.

Details

Reviewers

varun
tomek
ginsu
will

Commits

rCOMMc3e31c9faffb: [lib] Update union types in `message-types` to include `UpdateRelationship`…

Summary

Basically mirrored where the types in legacy-update-relationships were being consumed.

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

Depends on D11438

Test Plan

flow/things work as expected

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

atul created this revision.Mar 28 2024, 3:00 PM

Herald added a subscriber: ashoat. · View Herald TranscriptMar 28 2024, 3:00 PM

Harbormaster completed remote builds in B27770: Diff 38450.Mar 28 2024, 3:15 PM

atul requested review of this revision.Mar 28 2024, 3:15 PM

atul added a child revision: D11443: [lib] Update `messageContentFor[Client/Server]DB` to handle `UpdateRelationship`.Mar 28 2024, 4:59 PM

ginsu accepted this revision.Mar 29 2024, 3:25 AM

This revision is now accepted and ready to land.Mar 29 2024, 3:25 AM

land

Harbormaster completed remote builds in B27991: Diff 38795.Apr 4 2024, 7:53 PM

Closed by commit rCOMMc3e31c9faffb: [lib] Update union types in `message-types` to include `UpdateRelationship`…. · Explain WhyApr 4 2024, 8:00 PM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMMc3e31c9faffb: [lib] Update union types in `message-types` to include `UpdateRelationship`….

Revision Contents
Changeset List

Path

Size

lib/

types/

message-types.js

22 lines

Diff 38795

View Options

lib/types/message-types.js

[lib] Update union types in `message-types` to include `UpdateRelationship` typesClosedPublicActions