Page MenuHomePhabricator
Differential D12649

Projects - publish feed stories for project edits
ClosedPublic
Actions

Authored by btrahan on May 1 2015, 11:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 11:08 PM
Unknown Object (File)
Fri, Dec 13, 3:17 AM
Unknown Object (File)
Wed, Dec 11, 9:01 AM
Unknown Object (File)
Fri, Dec 6, 3:53 PM
Unknown Object (File)
Fri, Dec 6, 9:19 AM
Unknown Object (File)
Wed, Dec 4, 10:45 PM
Unknown Object (File)
Tue, Dec 3, 12:01 PM
Unknown Object (File)
Mon, Dec 2, 6:06 AM
View All Files
Subscribers
epriestley
Korvin
Tokens
"Mountain of Wealth" token, awarded by chad.

Details

Reviewers
epriestley
Maniphest Tasks
T7426: Creating a new project does not publish a feed story
Commits
Restricted Diffusion Commit
rP7aac1effacc6: Projects - publish feed stories for project edits
Summary

Fixes T7426. Wasn't 100% sure what the right feed notify phids were so I went with project subscribers.

Test Plan

made a project and saw the "btrahan created $project" story. edited project members and hashtags and got proper stories.

Diff Detail

Repository
rP Phabricator
Branch
T7426
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/project/storage/PhabricatorProjectTransaction.php:266XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 5661
Build 5680: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan updated this revision to Diff 30384.May 1 2015, 11:31 PM
btrahan added a task: T7426: Creating a new project does not publish a feed story.
btrahan retitled this revision from to Projects - publish feed stories for project edits.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptMay 1 2015, 11:31 PM
chad awarded a token.May 1 2015, 11:33 PM
epriestley accepted this revision.May 2 2015, 12:01 AM
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/project/editor/PhabricatorProjectTransactionEditor.php
412–419

I think this is redundant since Subscribers should put them in CC and the default is "To + Cc".

This revision is now accepted and ready to land.May 2 2015, 12:01 AM
btrahan updated this revision to Diff 30386.May 2 2015, 12:32 AM
btrahan edited edge metadata.

implement getMailTo (just return array()) so default getFeedNotifyPHIDs works

btrahan updated this revision to Diff 30387.May 2 2015, 12:33 AM

fix getMailTo signature (protected => public)

btrahan updated this revision to Diff 30388.May 2 2015, 12:34 AM

really get the signature right

Closed by commit rP7aac1effacc6: Projects - publish feed stories for project edits (authored by btrahan, committed by btrahan). · Explain WhyMay 2 2015, 12:34 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
project/
controller/
12 lines
editor/
15 lines
storage/
119 lines
infrastructure/
internationalization/
translation/
34 lines

Diff 30384

View Options

src/applications/project/controller/PhabricatorProjectEditDetailsController.php

Loading...
View Options

src/applications/project/editor/PhabricatorProjectTransactionEditor.php

Loading...
View Options

src/applications/project/storage/PhabricatorProjectTransaction.php

Loading...
View Options

src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php

Loading...
Phabricator · Built by Phacility