Page MenuHomePhabricator

[lib] Temporarily switch LOCAL to COMMUNITY_SECRET_SUBTHREAD for pending threads
ClosedPublic

Authored by ashoat on Jun 19 2024, 5:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 7, 1:21 AM
Unknown Object (File)
Feb 21 2025, 3:58 PM
Unknown Object (File)
Feb 3 2025, 12:16 PM
Unknown Object (File)
Feb 1 2025, 5:35 AM
Unknown Object (File)
Feb 1 2025, 5:35 AM
Unknown Object (File)
Feb 1 2025, 5:34 AM
Unknown Object (File)
Jan 27 2025, 8:08 PM
Unknown Object (File)
Jan 27 2025, 7:42 PM
Subscribers

Details

Summary

These two constants were hardcoded to be the same value. I used LOCAL instead of COMMUNITY_SECRET_SUBTHREAD in this case as a reminder to the author that we should be created thick threads here instead of thin threads eventually.

I'm ready to introduce a separate constant for LOCAL, but we'll need to do some additional work before we're able to switch to using it for new pending threads. For now I'll move this code to continue to use the same constant (COMMUNITY_SECRET_SUBTHREAD, which will stay as 4), but I'll return it back to using threadTypes.LOCAL once we're ready.

Depends on D12486

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable