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)
	Fri, Dec 13, 7:39 AM

	Unknown Object (File)
	Wed, Dec 11, 12:27 PM

	Unknown Object (File)
	Sat, Dec 7, 4:41 PM

	Unknown Object (File)
	Thu, Dec 5, 7:49 AM

	Unknown Object (File)
	Tue, Dec 3, 4:05 PM

	Unknown Object (File)
	Fri, Nov 29, 12:05 AM

	Unknown Object (File)
	Mon, Nov 25, 1:18 AM

	Unknown Object (File)
	Sat, Nov 23, 12:56 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...