mirror of
https://we.phorge.it/source/phorge.git
synced 2024-12-23 14:00:56 +01:00
Hide the "added a commit/revision" stories from feed and mail
Summary: Ref T13276. Previously, these edges were added directly with an `EdgeEditor`, so they did not generate transaction stories. Now, they're added properly, but they aren't terribly useful in feed/mail. Hide them in those contexts, like we already do with other types of similar edges. Test Plan: Will verify behavior on `secure`. Reviewers: amckinley Reviewed By: amckinley Maniphest Tasks: T13276 Differential Revision: https://secure.phabricator.com/D20491
This commit is contained in:
parent
3e0391c574
commit
104eefaa10
1 changed files with 4 additions and 0 deletions
|
@ -703,6 +703,8 @@ abstract class PhabricatorApplicationTransaction
|
|||
case DifferentialRevisionDependedOnByRevisionEdgeType::EDGECONST:
|
||||
case ManiphestTaskHasCommitEdgeType::EDGECONST:
|
||||
case DiffusionCommitHasTaskEdgeType::EDGECONST:
|
||||
case DiffusionCommitHasRevisionEdgeType::EDGECONST:
|
||||
case DifferentialRevisionHasCommitEdgeType::EDGECONST:
|
||||
return true;
|
||||
case PhabricatorProjectObjectHasProjectEdgeType::EDGECONST:
|
||||
// When an object is first created, we hide any corresponding
|
||||
|
@ -762,6 +764,8 @@ abstract class PhabricatorApplicationTransaction
|
|||
case DifferentialRevisionDependedOnByRevisionEdgeType::EDGECONST:
|
||||
case ManiphestTaskHasCommitEdgeType::EDGECONST:
|
||||
case DiffusionCommitHasTaskEdgeType::EDGECONST:
|
||||
case DiffusionCommitHasRevisionEdgeType::EDGECONST:
|
||||
case DifferentialRevisionHasCommitEdgeType::EDGECONST:
|
||||
return true;
|
||||
default:
|
||||
break;
|
||||
|
|
Loading…
Reference in a new issue