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
F3720891: D6932.id23413.diff
Wed, Jan 8, 12:55 PM
F3720890: D6932.id23368.diff
Wed, Jan 8, 12:55 PM
F3720889: D6932.id.diff
Wed, Jan 8, 12:55 PM
F3720888: D6932.diff
Wed, Jan 8, 12:55 PM
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
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