Page MenuHomePhabricator

[CommCoreModule] MessageSpec changes for updated `UPDATE_RELATIONSHIP`
ClosedPublic

Authored by atul on Mar 25 2024, 5:57 PM.
Tags
None
Referenced Files
F5095402: D11383.id38322.diff
Sun, Mar 30, 5:43 PM
Unknown Object (File)
Sun, Mar 30, 2:05 PM
Unknown Object (File)
Sun, Mar 30, 10:50 AM
Unknown Object (File)
Sun, Mar 30, 2:39 AM
Unknown Object (File)
Sat, Mar 29, 4:53 PM
Unknown Object (File)
Tue, Mar 4, 10:33 PM
Unknown Object (File)
Feb 18 2025, 8:49 PM
Unknown Object (File)
Feb 18 2025, 8:49 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable