From 9acc91d6917f88dd76e55abb3dde7b35683bb5b3 Mon Sep 17 00:00:00 2001 From: epriestley Date: Sat, 25 May 2013 08:46:06 -0700 Subject: [PATCH] Clear notifications about commits when viewing them Summary: We were missing this option on the response, so notifications about commits (e.g., tokens given) don't clear when viewing the commit. Test Plan: Gave a commit a token with user A, viewed notification with user B, viewed commit with user B and verified notification cleared. Reviewers: garoevans, btrahan Reviewed By: garoevans CC: aran Differential Revision: https://secure.phabricator.com/D6041 --- .../diffusion/controller/DiffusionCommitController.php | 1 + 1 file changed, 1 insertion(+) diff --git a/src/applications/diffusion/controller/DiffusionCommitController.php b/src/applications/diffusion/controller/DiffusionCommitController.php index ab781afa3e..2450857838 100644 --- a/src/applications/diffusion/controller/DiffusionCommitController.php +++ b/src/applications/diffusion/controller/DiffusionCommitController.php @@ -360,6 +360,7 @@ final class DiffusionCommitController extends DiffusionController { $content, array( 'title' => $commit_id, + 'pageObjects' => array($commit->getPHID()), 'dust' => true, )); }