Page MenuHomePhabricator
Differential D19972

Reorder "Merge" transaction to make "Close as Duplicate" produce a "[Merged]" email subject
ClosedPublic
Actions

Authored by epriestley on Jan 14 2019, 10:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 20, 4:27 PM
Unknown Object (File)
Thu, Apr 4, 6:36 PM
Unknown Object (File)
Wed, Apr 3, 4:07 PM
Unknown Object (File)
Sat, Mar 30, 4:42 PM
Unknown Object (File)
Sun, Mar 24, 9:30 PM
Unknown Object (File)
Mar 21 2024, 12:34 AM
Unknown Object (File)
Mar 20 2024, 10:54 PM
Unknown Object (File)
Mar 16 2024, 7:35 PM
View All 87 Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T11782: Inconsistent mail notification subject ([Updated] vs [Merged]) depending on from which task a duplicate task was merged
Commits
rPdc4d7f1f3e8d: Reorder "Merge" transaction to make "Close as Duplicate" produce a "[Merged]"…
Summary

Fixes T11782. When you "Close as Duplicate", generate a "[Merged]" email by making the merge the first transaction.

(There are other, more-deterministic ways to do this with action strength, but this is much simpler and I believe it suffices.)

Test Plan

Used "Close as Duplicate", got a "[Merged]" email out of it.

Diff Detail

Repository
rP Phabricator
Branch
mfa16
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21524
Build 29330: Run Core Tests
Build 29329: arc lint + arc unit

Revision Contents
Changeset List

PathSize
src/
applications/
maniphest/
relationship/
9 lines

Diff 47671

View Options

src/applications/maniphest/relationship/ManiphestTaskRelationship.php

Loading...
Phabricator · Built by Phacility