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

Authored by atul on Mar 28 2024, 8:00 AM.

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

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Mar 28 2024, 8:00 AM

Herald added a subscriber: ashoat. · View Herald TranscriptMar 28 2024, 8:00 AM

Harbormaster completed remote builds in B27770: Diff 38450.Mar 28 2024, 8:15 AM

atul requested review of this revision.Mar 28 2024, 8:15 AM

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

ginsu accepted this revision.Mar 28 2024, 8:25 PM

This revision is now accepted and ready to land.Mar 28 2024, 8:25 PM

land

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

Closed by commit rCOMMc3e31c9faffb: [lib] Update union types in `message-types` to include `UpdateRelationship`… (authored by atul). · Explain WhyApr 4 2024, 1: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 38802

View Options

lib/types/message-types.js

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