Page MenuHomePhabricator

Move notification-generating code to onRevisionDataUpdates
Open, Needs TriagePublic


  • Our existing Echo hook only fires for edits, RevisionDataUpdates fires also for changes to transclusions.
  • Right now CommentParser is run twice, once to generate notifications and again to deal with permalinks; this could potentially allow us to only parse once.
  • This would fix issues with notifications not being fired if we implemented per-page new topic notifications (T263821), and the change that added a topic only happened on a transcluded page