Page MenuHomePhabricator

Index and show Owners packages affected by Herald rules
ClosedPublic

Authored by epriestley on Apr 12 2019, 11:17 PM.
Tags
None
Referenced Files
F19067621: D20413.id.diff
Sun, Nov 30, 9:15 AM
F19059053: D20413.diff
Sat, Nov 29, 4:12 AM
F18865975: D20413.id48756.diff
Nov 3 2025, 2:52 PM
F18837254: D20413.diff
Oct 27 2025, 4:43 AM
F18834455: D20413.id.diff
Oct 26 2025, 8:48 AM
F18830128: D20413.diff
Oct 25 2025, 4:20 AM
F18803824: D20413.id48706.diff
Oct 18 2025, 4:11 AM
F18803812: D20413.id48756.diff
Oct 18 2025, 4:10 AM
Subscribers
Restricted Owners Package

Details

Summary

Depends on D20412. See PHI1147.

  • Index the targets of "Add Reviewer", "Add Blocking Reviewer", "Add Auditor", "Add Subscriber", and "Remove Subscriber" Herald rules. My major goal is to get Owners packages. This will also hit projects/users, but we just don't read those edges (for now, at least).
  • Add a "Related Herald Rules" panel to Owners Package pages.
  • Add a migration to reindex Herald rules for the recent build plan stuff and this, now that such a migration is easy to write.
Test Plan

Ran migration, verified all rules reindexed.

Screen Shot 2019-04-12 at 4.13.15 PM.png (988×1 px, 232 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable