Page MenuHomePhabricator

[keyserver] Update usage of threads.default_role in thread-fetchers.js
ClosedPublic

Authored by rohan on Nov 24 2023, 9:57 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 13, 6:53 PM
Unknown Object (File)
Wed, Nov 13, 1:33 AM
Unknown Object (File)
Sat, Nov 9, 2:41 AM
Unknown Object (File)
Sat, Nov 9, 2:40 AM
Unknown Object (File)
Sat, Nov 9, 2:37 AM
Unknown Object (File)
Sat, Nov 9, 1:29 AM
Unknown Object (File)
Oct 27 2024, 1:47 PM
Unknown Object (File)
Oct 25 2024, 5:17 PM
Subscribers

Details

Summary

We need to get rid of usages of threads.default_role throughout the codebase so we can eventually drop the column.

This diff handles the usage in thread-fetchers.js

Existing code: Fetches the default_role when constructed threadInfos to set the isDefault flag for each role in threadInfo.roles

Depends on D9979

Addresses part of ENG-5833

Test Plan

Updated my query and logged the threadInfo.roles. Confirmed that for each role, isDefault was set to true for all Members and false otherwise.

Another test I did was to compare the logged threadInfo.roles both before and after the query change and they were identical

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable