Differential D10951

Projects - fix missing subscriber handling code in the editor
ClosedPublic
Actions

Authored by btrahan on Dec 9 2014, 6:13 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Apr 23, 5:29 PM

	Unknown Object (File)
	Sun, Apr 21, 10:28 AM

	Unknown Object (File)
	Fri, Apr 19, 5:02 PM

	Unknown Object (File)
	Mon, Apr 15, 5:30 PM

	Unknown Object (File)
	Thu, Apr 11, 5:48 PM

	Unknown Object (File)
	Sun, Apr 7, 8:26 PM

	Unknown Object (File)
	Wed, Apr 3, 5:08 PM

	Unknown Object (File)
	Tue, Apr 2, 8:18 PM

Subscribers

Tokens

"Orange Medal" token, awarded by epriestley.

Details

Reviewers

Maniphest Tasks

T6719: Projects are missing an internal apply implementation for edge transactions

Commits

Restricted Diffusion Commit
rP5b59a81ab9b3: Projects - fix missing subscriber handling code in the editor

Summary

Fixes T6719. At some point, we added automagical subscriptions via @mentions, and these were failing in project descriptions from a lack of an implementation in the editor. Said "implementation" is to do nothing, but it needs to be there nonetheless.

Test Plan

updated a project mentioning someone in the description and it worked. also saw a 'subscriber added' transaction

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

btrahan updated this revision to Diff 26298.Dec 9 2014, 6:13 PM

btrahan added a task: T6719: Projects are missing an internal apply implementation for edge transactions.

btrahan retitled this revision from to Projects - fix missing subscriber handling code in the editor.

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 TranscriptDec 9 2014, 6:13 PM

epriestley accepted this revision.Dec 9 2014, 6:13 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Dec 9 2014, 6:13 PM

epriestley awarded a token.Dec 9 2014, 6:14 PM

Closed by commit rP5b59a81ab9b3: Projects - fix missing subscriber handling code in the editor (authored by btrahan, committed by btrahan). · Explain WhyDec 9 2014, 6:14 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

project/

editor/

PhabricatorProjectTransactionEditor.php

2 lines

Diff 26299

src/applications/project/editor/PhabricatorProjectTransactionEditor.php

Loading...