Page MenuHomePhabricator

[CommCoreModule] MessageSpec changes for updated `UPDATE_RELATIONSHIP`
ClosedPublic

Authored by atul on Mar 25 2024, 5:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Oct 11, 10:35 AM
Unknown Object (File)
Fri, Oct 11, 10:35 AM
Unknown Object (File)
Fri, Oct 11, 10:35 AM
Unknown Object (File)
Fri, Oct 11, 10:34 AM
Unknown Object (File)
Fri, Oct 11, 9:15 AM
Unknown Object (File)
Sep 12 2024, 8:29 AM
Unknown Object (File)
Sep 5 2024, 9:23 PM
Unknown Object (File)
Sep 5 2024, 9:23 PM
Subscribers

Details

Summary

This is effectively https://github.com/CommE2E/comm/commit/3d47f80c594a69206179fd8f33bb57d31b1fd78d and https://github.com/CommE2E/comm/commit/f1debdb595bcba76db8e91ff8d70519b9b0a785b squashed together into a single diff.

We use the same spec for both on the C++ side.


Depends on D11382

Test Plan

Close reading + builds

NOTE: It appears that we're missing a spec for TOGGLE_PIN. This doesn't appear to cause any issues on C++ side since message_specs_initializer is a Map rather than eg std::vector so it's fine that we have a "gap."

Diff Detail

Repository
rCOMM Comm
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage