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
Unknown Object (File)
Fri, Nov 22, 12:23 PM
Unknown Object (File)
Thu, Nov 21, 12:31 AM
Unknown Object (File)
Sat, Nov 16, 11:22 PM
Unknown Object (File)
Sun, Nov 10, 8:14 AM
Unknown Object (File)
Sat, Nov 9, 12:45 AM
Unknown Object (File)
Fri, Nov 8, 11:21 PM
Unknown Object (File)
Fri, Nov 8, 9:44 PM
Unknown Object (File)
Fri, Nov 8, 9:43 PM
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