Diffusion Phabricator 7b50eef27a56

Special case the "added projects" transaction in mail when creating objects
7b50eef27a56
Actions

Description

Special case the "added projects" transaction in mail when creating objects

Summary: Fixes T10493. See that task and inline comments for discussion.

Test Plan:
Created an object with some projects, saw the transaction in resulting mail:

Screen Shot 2016-05-18 at 6.57.02 AM.png (315×508 px, 39 KB)

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T10493

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

Details

Provenance

epriestley	Authored on May 18 2016, 1:55 PM
epriestley	Pushed on May 18 2016, 2:09 PM

Reviewer

chad

Differential Revision

D15942: Special case the "added projects" transaction in mail when creating objects

Parents

rP3aed39b8b072: Fix an issue with serializing reviewers over the wire

Branches

Unknown

Tags

Unknown

Tasks

T10493: Task creation no longer lists projects added (neither the UI nor mail notifications)

Build Status

Buildable 12256
Build 15486: Run Core Tests

Event Timeline

epriestley committed rP7b50eef27a56: Special case the "added projects" transaction in mail when creating objects (authored by epriestley).May 18 2016, 2:09 PM

epriestley added a task: T10493: Task creation no longer lists projects added (neither the UI nor mail notifications).

Harbormaster completed building B12256: rP7b50eef27a56: Special case the "added projects" transaction in mail when creating objects.May 18 2016, 2:10 PM

adelabreteche mentioned this in T10988: Unable to Import an Existing External Repository .May 19 2016, 7:58 AM

Changes (1)

Path

Size

src/

applications/

transactions/

storage/

PhabricatorApplicationTransaction.php

rP7b50eef27a56

View Options

src/applications/transactions/storage/PhabricatorApplicationTransaction.php