Page MenuHomePhabricator
Differential D8861

Email paste link
ClosedPublic
Actions

Authored by rugabarbo on Apr 25 2014, 6:40 PM.
Tags
None
Referenced Files
F13172496: D8861.diff
Tue, May 7, 4:00 PM
F13168295: D8861.diff
Tue, May 7, 7:38 AM
Unknown Object (File)
Sun, May 5, 9:11 AM
Unknown Object (File)
Fri, May 3, 6:27 AM
Unknown Object (File)
Sun, Apr 28, 8:23 PM
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
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
  1. Go to paste
  2. Add comment
  3. Check that paste subsribers got paste link by email

See T4878 for more details.

Diff Detail

Repository
rP Phabricator
Branch
email-paste-link
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 68
Build 68: [Placeholder Plan] Wait for 30 Seconds

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
epriestley accepted this revision.Apr 25 2014, 6:50 PM
epriestley edited edge metadata.

Thanks!

This revision is now accepted and ready to land.Apr 25 2014, 6:50 PM
epriestley closed this revision.Apr 25 2014, 6:50 PM
epriestley updated this revision to Diff 21020.

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

Revision Contents
Changeset List

PathSize
src/
applications/
paste/
editor/
13 lines

Diff 21019

View Options

src/applications/paste/editor/PhabricatorPasteEditor.php

Loading...
Phabricator · Built by Phacility