Page MenuHomePhabricator

Fix an issue where some Differential edit pathways may not have reviewers attached
ClosedPublic

Authored by epriestley on Feb 7 2018, 5:42 PM.
Tags
None
Referenced Files
F15500867: D19022.id45617.diff
Sun, Apr 13, 7:41 PM
F15454070: D19022.id45617.diff
Sat, Mar 29, 4:07 PM
F15448017: D19022.id45591.diff
Fri, Mar 28, 2:53 AM
F15447237: D19022.id45617.diff
Thu, Mar 27, 10:40 PM
F15436722: D19022.id.diff
Tue, Mar 25, 4:13 PM
F15431383: D19022.diff
Mar 24 2025, 12:02 PM
F15422823: D19022.id45591.diff
Mar 22 2025, 10:58 AM
F15394031: D19022.id.diff
Mar 15 2025, 11:19 PM
Subscribers
None

Details

Summary

Depends on D19021. Ref T13053. When you "Subscribe", or make some other types of edits, we don't necessarily have reviewer data, but may now need it to do the new recipient list logic.

I don't have a totally clean way to deal with this in the general case in mind, but just load it for now so that things don't fatal.

Test Plan

Subscribed to a revision with the "Subscribe" action.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable