feat: support watched term alerts for the mod team - #5145
Conversation
Greptile SummaryAdds moderator-managed watched terms and Discord alerts for matching forum posts and comments.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains. Reviews (3): Last reviewed commit: "fix: address feedback" | Re-trigger Greptile |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
| }; | ||
| } | ||
|
|
||
| public function isManagementComment(): bool |
There was a problem hiding this comment.
Would isUmoderatedComment be better?
There was a problem hiding this comment.
Is there any reason to have a created_by field or some form of audit log? There's nothing indicating who added a word (or changed one to something else).
| 'inside a URL' => 'see https://example.com/watchedtool/downloads', | ||
| 'split by bold tags' => 'grab watched[b][/b]tool here', | ||
| 'split by italic tags' => 'grab watched[i][/i]tool here', | ||
| 'split by a url tag' => 'grab watched[url=https://example.com][/url]tool here', |
There was a problem hiding this comment.
What about in a url tag?
grab it [url=https://watchedtool.com]here[/url]
There was a problem hiding this comment.
On a related note: Since the minimum term length is four characters, are we at all concerned about falsely matching a substring in a hash like YouTube videos and bit.ly links generate?
2 participants
Resolves #5091.
https://discord.com/channels/476211979464343552/1488680676021768414/1533221430324629614
Watched terms can now be configured by users with
Role::MODERATORattached. When a given term is written on a forum post or comment, it will surface in the moderator feed.