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
F3392199: D7078.diff
Sat, Nov 30, 7:30 AM
Unknown Object (File)
Fri, Nov 22, 12:19 AM
Unknown Object (File)
Fri, Nov 22, 12:19 AM
Unknown Object (File)
Fri, Nov 22, 12:19 AM
Unknown Object (File)
Fri, Nov 22, 12:18 AM
Unknown Object (File)
Mon, Nov 11, 12:04 AM
Unknown Object (File)
Mon, Nov 4, 11:34 PM
Unknown Object (File)
Mon, Nov 4, 11:34 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
No Lint Coverage
Unit
No Test Coverage