Page MenuHomePhabricator

[keyserver] Only set sidebarCreation flag for the first message in the sidebar
ClosedPublic

Authored by ashoat on Mar 2 2023, 2:01 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 15, 6:27 PM
Unknown Object (File)
Sun, Dec 15, 6:27 PM
Unknown Object (File)
Sun, Dec 15, 6:26 PM
Unknown Object (File)
Sun, Dec 15, 6:19 PM
Unknown Object (File)
Dec 4 2024, 5:43 PM
Unknown Object (File)
Dec 3 2024, 1:02 AM
Unknown Object (File)
Dec 1 2024, 9:40 PM
Unknown Object (File)
Dec 1 2024, 7:40 PM
Subscribers

Details

Summary

When I say "first message" in the commit title, I really mean the third message, as each sidebar starts with a SIDEBAR_SOURCE and then a CREATE_SIDEBAR.

Check out the code comment for an explanation as to why we're doing this.

Test Plan

Force sidebarCreation to true on the client for all text messages, and confirm that it is ignored when invalid

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable