Page MenuHomePhabricator

Stop sending Phriction edit mail to the previous/current content author
ClosedPublic

Authored by epriestley on Feb 27 2018, 10:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 1:44 AM
Unknown Object (File)
Fri, Apr 5, 4:52 PM
Unknown Object (File)
Wed, Apr 3, 12:15 AM
Unknown Object (File)
Feb 28 2024, 2:04 PM
Unknown Object (File)
Feb 28 2024, 1:23 PM
Unknown Object (File)
Feb 9 2024, 6:03 PM
Unknown Object (File)
Dec 25 2023, 5:07 PM
Unknown Object (File)
Dec 21 2023, 1:10 AM
Subscribers
None

Details

Summary

See https://discourse.phabricator-community.org/t/previous-author-of-phriction-page-always-receives-mail-when-new-author-edits-the-page/1155.

After the "send users email when they are removed from the reviewer list" change which includes old To/Cc on the list, Phriction now emails the previous editor.

This is weird and unintended. The behavior isn't necessarily always bad, but if it's sometimes desirable we should make it explicit.

For now, just get rid of the weird side effect.

Test Plan

Edited a Phriction page.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Feb 27 2018, 11:05 PM
This revision was automatically updated to reflect the committed changes.