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
F15536397: D14171.id34288.diff
Thu, Apr 24, 9:29 AM
F15506031: D14171.diff
Tue, Apr 15, 8:01 AM
F15499263: D14171.id34240.diff
Sun, Apr 13, 2:46 PM
F15488421: D14171.id34240.diff
Thu, Apr 10, 10:12 PM
F15488293: D14171.id34288.diff
Thu, Apr 10, 9:02 PM
F15484289: D14171.id.diff
Wed, Apr 9, 5:26 PM
F15480415: D14171.diff
Tue, Apr 8, 2:01 PM
F15443480: D14171.id.diff
Mar 27 2025, 4:38 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