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
F14648187: D19022.id45617.diff
Sat, Jan 11, 2:02 PM
Unknown Object (File)
Wed, Jan 8, 7:22 AM
Unknown Object (File)
Sun, Jan 5, 6:21 PM
Unknown Object (File)
Mon, Dec 30, 1:04 AM
Unknown Object (File)
Sun, Dec 29, 11:04 PM
Unknown Object (File)
Sat, Dec 21, 11:14 PM
Unknown Object (File)
Fri, Dec 20, 4:54 AM
Unknown Object (File)
Dec 10 2024, 9:32 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
Branch
mailer19
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19385
Build 26220: Run Core Tests
Build 26219: arc lint + arc unit