Page MenuHomePhabricator
Differential D13905

Add ability to reply to Ponder Answer emails
ClosedPublic
Actions

Authored by chad on Aug 15 2015, 1:27 AM.
Tags
None
Referenced Files
F19862941: D13905.id33563.diff
Sat, Mar 14, 10:53 AM
F19859225: D13905.diff
Sat, Mar 14, 2:56 AM
F19853570: D13905.id33562.diff
Fri, Mar 13, 6:35 AM
F19817829: D13905.id33604.diff
Mar 5 2026, 3:54 PM
F19810078: D13905.id.diff
Mar 3 2026, 5:12 PM
F19733158: D13905.diff
Feb 14 2026, 9:34 PM
F19645906: D13905.id33562.diff
Feb 6 2026, 12:47 AM
F19610903: D13905.id33604.diff
Feb 3 2026, 5:14 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T3846: Build reply handler functionality for Ponder
T9068: Answers don't publish to feed/mail
Commits
Restricted Diffusion Commit
rPee8e4363900e: Add ability to reply to Ponder Answer emails
Summary

Ref T9068, Ref T3846. Maybe fixes both, but I'm having issues testing email replies in a sandbox. Moves answer feed/mail generation to the AnswerEditor, hides it in QuestionEditor.

Test Plan

Write an answer, see feed story, check /mail/ for mail generation.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 33562.Aug 15 2015, 1:27 AM
chad retitled this revision from to Add ability to reply to Ponder Answer emails.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
chad added tasks: T9068: Answers don't publish to feed/mail, T3846: Build reply handler functionality for Ponder.
Herald added a subscriber: Korvin. · View Herald TranscriptAug 15 2015, 1:27 AM
chad updated this revision to Diff 33563.Aug 15 2015, 1:40 AM
  • remove custom feed handler
epriestley accepted this revision.Aug 17 2015, 4:54 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Aug 17 2015, 4:54 PM
Closed by commit rPee8e4363900e: Add ability to reply to Ponder Answer emails (authored by chad, committed by chad). · Explain WhyAug 17 2015, 5:22 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 33604

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/ponder/editor/PonderQuestionEditor.php

Loading...
View Options

src/applications/ponder/feed/PonderTransactionFeedStory.php

Loading...
View Options

src/applications/ponder/mail/PonderAnswerMailReceiver.php

Loading...
View Options

src/applications/ponder/mail/PonderAnswerReplyHandler.php

Loading...
View Options

src/applications/ponder/storage/PonderQuestionTransaction.php

Loading...
Phabricator · Built by Phacility