Homec4science

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

Authored by epriestley <git@epriestley.com> on Feb 29 2016, 01:28.

Description

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

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.)

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T10466

Differential Revision: https://secure.phabricator.com/D15361

Details

Committed
epriestley <git@epriestley.com>Feb 29 2016, 01:32
Pushed
aubortJan 31 2017, 17:16
Parents
rPH5295c6ba1e61: Use EditEngine for Harbormaster Build Plans, fix some crumbs/mobile stuff
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH2e19b78b8dd6: Don't publish anything for "Disable Mail" / "Enable Mail" on projects (authored by epriestley <git@epriestley.com>).Feb 29 2016, 01:32