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

	F14083016: D10951.diff
	Sat, Nov 23, 2:33 AM

	Unknown Object (File)
	Wed, Nov 6, 1:17 AM

	Unknown Object (File)
	Fri, Oct 25, 1:50 PM

	Unknown Object (File)
	Oct 21 2024, 4:16 PM

	Unknown Object (File)
	Oct 16 2024, 12:01 PM

	Unknown Object (File)
	Oct 11 2024, 9:50 PM

	Unknown Object (File)
	Aug 30 2024, 2:45 AM

	Unknown Object (File)
	Aug 22 2024, 7:16 AM

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...