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
Unknown Object (File)
Wed, Nov 13, 1:19 AM
Unknown Object (File)
Wed, Nov 13, 12:05 AM
Unknown Object (File)
Wed, Nov 13, 12:05 AM
Unknown Object (File)
Wed, Nov 13, 12:05 AM
Unknown Object (File)
Wed, Nov 13, 12:05 AM
Unknown Object (File)
Wed, Nov 13, 12:05 AM
Unknown Object (File)
Wed, Nov 13, 12:05 AM
Unknown Object (File)
Wed, Nov 13, 12:05 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
Branch
identity-search-land
Lint
No Lint Coverage
Unit
No Test Coverage

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.