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.
Details
Details
- Reviewers
chad - Maniphest Tasks
- T10466: Wrong mail and extra feed story when joining a project
- Commits
- Restricted Diffusion Commit
rP2e19b78b8dd6: Don't publish anything for "Disable Mail" / "Enable Mail" on projects
- 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
Diff Detail
- Repository
- rP Phabricator
- Branch
- nomaildisable
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 10878 Build 13415: Run Core Tests Build 13414: arc lint + arc unit