Page MenuHomePhabricator

Don't publish anything for "Disable Mail" / "Enable Mail" on projects
ClosedPublic

Authored by epriestley on Feb 29 2016, 12:31 AM.
Tags
None
Referenced Files
F14059619: D15361.id37038.diff
Sun, Nov 17, 8:42 PM
F14008110: D15361.diff
Tue, Oct 29, 3:52 PM
F14006913: D15361.id37039.diff
Mon, Oct 28, 9:00 PM
Unknown Object (File)
Oct 4 2024, 3:49 AM
Unknown Object (File)
Oct 3 2024, 3:52 PM
Unknown Object (File)
Sep 28 2024, 8:17 PM
Unknown Object (File)
Sep 9 2024, 6:52 AM
Unknown Object (File)
Sep 1 2024, 4:09 PM
Subscribers
None

Details

Summary

Fixes T10466. Currently, clicking "Disable Mail" or "Enable Mail" on a project toggles an edge, but it gets a default "added an edge" story and transaction record.

These are confusing, useless and not interesting, so just hide them.

Test Plan
  • Before patch: clicked enable/disable mail, saw "added an edge" / "removed an edge" stories in feed and project history.
  • After patch: clicked enable/disable mail, saw nothing in feed or project history.
  • (Note that this patch is not retroactive for feed, so already-published stories won't unpublish.)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Don't publish anything for "Disable Mail" / "Enable Mail" on projects.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Feb 29 2016, 12:32 AM
This revision was automatically updated to reflect the committed changes.