Page MenuHomePhabricator
Differential D14171

Make Ponder Emails a little more consistently delivered
ClosedPublic
Actions

Authored by chad on Sep 26 2015, 8:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Feb 17, 12:44 AM
Unknown Object (File)
Fri, Feb 14, 3:42 AM
Unknown Object (File)
Thu, Feb 13, 7:56 AM
Unknown Object (File)
Mon, Feb 10, 9:29 AM
Unknown Object (File)
Sat, Feb 8, 11:22 PM
Unknown Object (File)
Sat, Feb 8, 2:01 PM
Unknown Object (File)
Sat, Feb 1, 12:37 AM
Unknown Object (File)
Fri, Jan 31, 8:46 AM
View All Files
Subscribers
Korvin
revi

Details

Reviewers
epriestley
Maniphest Tasks
T9271: ponder (sometimes?) does not send email when an answer is added
Commits
Restricted Diffusion Commit
rPae082c60338d: Make Ponder Emails a little more consistently delivered
Summary

Ref T9271, maybe fixes it. This restores feed publishing for answers (broken in D13951) and sends the author of the question an email for new answers. Also, unsure how to pull all question subsribers to the answer email, or is it automagical?

Test Plan

notchad asks a question, chad answers, log into notchad and see that mail was delivered, see feed story.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 34240.Sep 26 2015, 8:08 PM
chad retitled this revision from to Make Ponder Emails a little more consistently delivered.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
chad added a task: T9271: ponder (sometimes?) does not send email when an answer is added.
Herald added a subscriber: Korvin. · View Herald TranscriptSep 26 2015, 8:08 PM
revi added a subscriber: revi.Sep 27 2015, 2:28 AM
epriestley accepted this revision.Sep 29 2015, 5:06 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Sep 29 2015, 5:06 PM
Closed by commit rPae082c60338d: Make Ponder Emails a little more consistently delivered (authored by chad, committed by chad). · Explain WhySep 29 2015, 9:25 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
ponder/
editor/
21 lines
7 lines
7 lines

Diff 34288

View Options

src/applications/ponder/editor/PonderAnswerEditor.php

Loading...
View Options

src/applications/ponder/editor/PonderEditor.php

Loading...
View Options

src/applications/ponder/editor/PonderQuestionEditor.php

Loading...
Phabricator · Built by Phacility