Email paste link
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Apr 25, 2:38 PM

	Unknown Object (File)
	Wed, Apr 24, 10:59 PM

	Unknown Object (File)
	Sat, Apr 20, 6:48 PM

	Unknown Object (File)
	Wed, Apr 17, 3:16 PM

	Unknown Object (File)
	Sat, Apr 6, 11:12 PM

	Unknown Object (File)
	Tue, Apr 2, 4:25 PM

	Unknown Object (File)
	Mon, Apr 1, 10:55 PM

	Unknown Object (File)
	Mar 21 2024, 8:17 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