Page MenuHomePhabricator

Stop "Mute Notifications" on Bulk Jobs from fataling

Authored by epriestley on Feb 28 2019, 5:48 PM.



See Bulk Jobs have an "edge" table but currently do not support edge transactions. Add support.

This stops "Mute Notifications" from fataling.

The action probably doesn't do what the reporting user expects (it stops edits to the job object from sending notifications; it does not stop the edits the job performs from sending notifications) but I think this change puts us in a better place no matter what, even if we eventually clarify or remove this behavior.

Test Plan

Clicked "Mute Notifications" on a bulk job, got an effect instead of a fatal.

Diff Detail

rP Phabricator
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Feb 28 2019, 5:48 PM
epriestley requested review of this revision.Feb 28 2019, 5:50 PM
amckinley accepted this revision.Mar 1 2019, 12:19 AM
This revision is now accepted and ready to land.Mar 1 2019, 12:19 AM
This revision was automatically updated to reflect the committed changes.