Page MenuHomePhabricator

Separate "feed" and "notifications" better, allow stories to appear in notifications only
ClosedPublic

Authored by epriestley on Dec 10 2018, 10:23 PM.
Tags
None
Referenced Files
F18828419: D19864.id.diff
Fri, Oct 24, 4:09 PM
F18824951: D19864.diff
Thu, Oct 23, 6:47 PM
F18760175: D19864.diff
Mon, Oct 6, 8:21 AM
F18746778: D19864.id47441.diff
Fri, Oct 3, 1:26 PM
F18722369: D19864.id47431.diff
Mon, Sep 29, 10:33 PM
F18612231: D19864.diff
Sep 14 2025, 8:58 AM
F18525721: D19864.id.diff
Sep 6 2025, 3:41 PM
F18509993: D19864.id.diff
Sep 5 2025, 3:43 AM
Subscribers
None

Details

Summary

Depends on D19861. Ref T13222. See PHI996. Fixes T10743. Currently, notifications only work if a story also has a feed rendering.

Separate "visible in feed" and "visible in notifications", and make notifications query only notifications and vice versa.

Then, set the test notification stories to be visible in notifications only, not feed.

This could be refined a bit (there's no way to have the two views render different values today, for example) but since the only actual use case we have right now is test notifications I don't want to go too crazy future-proofing it. I could imagine doing some more of this kind of stuff in Conpherence eventually, though, perhaps.

Test Plan

Sent myself test notifications, saw them appear on my profile timeline and in the JS popup, and in my notifications menu, but not in feed.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable