Email paste link
ClosedPublic
Actions

Authored by rugabarbo on Apr 25 2014, 6:40 PM.

Tags

None

Referenced Files

	F14061779: D8861.diff
	Mon, Nov 18, 8:34 AM

	F14048812: D8861.diff
	Thu, Nov 14, 9:49 AM

	F14033504: D8861.diff
	Sat, Nov 9, 5:50 PM

	F14033353: D8861.id21020.diff
	Sat, Nov 9, 5:31 PM

	F14033352: D8861.id21019.diff
	Sat, Nov 9, 5:31 PM

	F14033351: D8861.id.diff
	Sat, Nov 9, 5:31 PM

	F14021742: D8861.diff
	Wed, Nov 6, 11:53 AM

	F14020534: D8861.diff
	Wed, Nov 6, 12:41 AM

View All Files

Subscribers

epriestley

Korvin

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T4878: Paste notification doesn't contain link to it

Commits

Restricted Diffusion Commit
rP5e80901cecf0: Email paste link

Summary

Fixes T4878

Test Plan

Go to paste
Add comment
Check that paste subsribers got paste link by email

See T4878 for more details.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

rugabarbo updated this revision to Diff 21019.Apr 25 2014, 6:40 PM

rugabarbo retitled this revision from to Email paste link.

rugabarbo updated this object.

rugabarbo edited the test plan for this revision. (Show Details)

rugabarbo added a reviewer: epriestley.

rugabarbo added a task: T4878: Paste notification doesn't contain link to it.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptApr 25 2014, 6:40 PM

Herald added subscribers: Korvin, epriestley. · View Herald Transcript

Harbormaster completed remote builds in B68: Diff 21019.Apr 25 2014, 6:41 PM

Thanks!

This revision is now accepted and ready to land.Apr 25 2014, 6:50 PM

Closed by commit rP5e80901cecf0 (authored by @rugabarbo, committed by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

paste/

editor/

PhabricatorPasteEditor.php

13 lines

Diff 21020

View Options

src/applications/paste/editor/PhabricatorPasteEditor.php

Email paste linkClosedPublicActions