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

	F18097240: D10951.id26299.diff
	Fri, Aug 8, 12:21 PM

	F18093184: D10951.id.diff
	Thu, Aug 7, 4:08 PM

	F18090692: D10951.diff
	Wed, Aug 6, 5:04 PM

	Unknown Object (File)
	May 6 2025, 10:30 PM

	Unknown Object (File)
	Apr 24 2025, 2:19 AM

	Unknown Object (File)
	Apr 21 2025, 10:45 AM

	Unknown Object (File)
	Apr 20 2025, 3:54 PM

	Unknown Object (File)
	Apr 19 2025, 2:26 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

Branch

T6719

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 3203
Build 3209: [Placeholder Plan] Wait for 30 Seconds

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 26298

src/applications/project/editor/PhabricatorProjectTransactionEditor.php

Loading...