Page MenuHomePhabricator

[identity] [shared] Separate identity search messages into client and server messages
ClosedPublic

Authored by will on Feb 5 2024, 7:40 AM.
Tags
None
Referenced Files
F3486734: D10945.id37619.diff
Wed, Dec 18, 5:49 AM
F3486680: D10945.id36867.diff
Wed, Dec 18, 5:41 AM
F3486558: D10945.id37277.diff
Wed, Dec 18, 5:16 AM
F3486407: D10945.id37364.diff
Wed, Dec 18, 4:52 AM
F3486085: D10945.diff
Wed, Dec 18, 2:53 AM
Unknown Object (File)
Tue, Dec 17, 12:55 AM
Unknown Object (File)
Tue, Dec 17, 12:55 AM
Unknown Object (File)
Tue, Dec 17, 12:55 AM
Subscribers

Details

Summary

This separates client and server messages into two different enums for organization purposes. This will be reflected in the client javascript flow types for proper runtime checking.

Depends on D10981

Test Plan

cargo build

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

will retitled this revision from [identity] [shared] Separate identity search messages into client and server messages to [identity] [shared] [1/n] Separate identity search messages into client and server messages.Feb 5 2024, 7:42 AM
will requested review of this revision.Feb 5 2024, 8:00 AM
will planned changes to this revision.Feb 5 2024, 11:38 AM
will retitled this revision from [identity] [shared] [1/n] Separate identity search messages into client and server messages to [identity] [shared] [3/n] Separate identity search messages into client and server messages.
will retitled this revision from [identity] [shared] [3/n] Separate identity search messages into client and server messages to [identity] [shared] Separate identity search messages into client and server messages.Feb 8 2024, 10:13 AM
varun added inline comments.
shared/identity_search_messages/src/messages/mod.rs
2 ↗(On Diff #37014)

can we add a note here like we did in D10928 to make sure this file stays in sync with lib/types/identity-search/messages.js?

This revision is now accepted and ready to land.Feb 13 2024, 11:21 AM
This revision was landed with ongoing or failed builds.Feb 26 2024, 12:45 PM
This revision was automatically updated to reflect the committed changes.