Page MenuHomePhabricator

Update the "Notification Test" workflow to use more modern mechanisms
ClosedPublic

Authored by epriestley on Dec 10 2018, 8:35 PM.
Tags
None
Referenced Files
F19491502: D19861.id47424.diff
Thu, Jan 8, 7:22 AM
F19183191: D19861.id47424.diff
Tue, Dec 16, 4:02 AM
F19059644: D19861.id.diff
Nov 29 2025, 6:25 AM
F19055139: D19861.diff
Nov 28 2025, 3:53 PM
F19002804: D19861.diff
Nov 21 2025, 6:57 AM
F18837584: D19861.id.diff
Oct 27 2025, 6:37 AM
F18835480: D19861.diff
Oct 26 2025, 4:06 PM
F18804720: D19861.id47424.diff
Oct 18 2025, 11:20 AM
Subscribers
None

Details

Summary

Depends on D19860. Ref T13222. Ref T10743. See PHI996.

Long ago, there were different types of feed stories. Over time, there was less and less need for this, and nowadays basically everything is a "transaction" feed story. Each story renders differently, but they're fundamentally all about transactions.

The Notification test controller still uses a custom type of feed story to send notifications. Move away from this, and apply a transaction against the user instead. This has the same ultimate effect, but involves less weird custom code from ages long forgotten.

This doesn't fix the actual problem with these things showing up in feed. Currently, stories always use the same rendering for feed and notifications, and there need to be some additional changes to fix this. So no behavioral change yet, just slightly more reasonable code.

Test Plan

Clicked the button and got some test notifications, with Aphlict running.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable