Page MenuHomePhabricator

[lib] Pass threadType and parentThreadID to alwaysDisplayShortName ThreadEntity
ClosedPublic

Authored by ashoat on Mar 15 2023, 12:15 PM.
Tags
None
Referenced Files
F4260908: D7078.id23766.diff
Mon, Feb 24, 12:52 PM
F4260907: D7078.id.diff
Mon, Feb 24, 12:52 PM
Unknown Object (File)
Sun, Feb 23, 3:51 AM
Unknown Object (File)
Tue, Feb 18, 2:45 PM
Unknown Object (File)
Mon, Feb 3, 11:49 AM
Unknown Object (File)
Jan 20 2025, 5:17 AM
Unknown Object (File)
Jan 17 2025, 1:27 PM
Unknown Object (File)
Jan 10 2025, 2:47 PM
Subscribers

Details

Summary

We need these params for threadNoun to be able to accurately pick a noun for thread.

We already pass threadType (when available) to threadNoun, so including that should "automatically" improve things.

parentThreadID is new, as is necessary to differentiate children of GENESIS from children of other communities. The following diff will update threadNoun to look at that.

Test Plan

Tested in combination with the next two diffs:

  1. Make sure that robotext for chat creation of GENESIS subchannels calls them "chats"
  2. Make sure that robotext for chat creation of non-GENESIS subchannels calls them "channels"
  3. Make sure that notifs for chat creation of GENESIS subchannels calls them "chats"
  4. Make sure that notifs for chat creation of GENESIS subchannels doesn't mention GENESIS

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable