Page MenuHomePhabricator

Add Ponder Question mail create receiver
ClosedPublic

Authored by chad on Jun 8 2016, 8:42 PM.
Tags
None
Referenced Files
F12833621: D16084.id38699.diff
Thu, Mar 28, 2:09 PM
Unknown Object (File)
Thu, Mar 21, 11:51 AM
Unknown Object (File)
Thu, Mar 21, 11:50 AM
Unknown Object (File)
Feb 12 2024, 9:06 PM
Unknown Object (File)
Feb 12 2024, 9:05 PM
Unknown Object (File)
Feb 12 2024, 8:52 PM
Unknown Object (File)
Feb 12 2024, 3:26 PM
Unknown Object (File)
Feb 8 2024, 6:13 PM
Subscribers

Details

Summary

Fixes T11115, but unclear how to test this. I think I've asked this in the past.

Test Plan
  • Visit Applications -> Ponder
  • Configure external email
  • Test External Email
  • See new Question

Diff Detail

Repository
rP Phabricator
Branch
ponder-create-mail (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12571
Build 15953: Run Core Tests
Build 15952: arc lint + arc unit

Event Timeline

chad retitled this revision from to Add Ponder Question mail create receiver.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.

I think you can test this with bin/mail receive-test. Something like this:

phabricator/ $ echo "test" | ./bin/mail receive-test --as chad --to ponder@whatever.com

Give that a shot? We might need to add a --subject flag too, but that should theoretically work. If you don't get anywhere with it, let me know and I'll see if I can fix it so it works.

Code itself looks good to me.

src/applications/ponder/mail/PonderQuestionCreateMailReceiver.php
21

Incredibly minor, but this will prevent creating questions with great titles like "0" and "0.0". Doing if (!strlen($title)) instead will work for all titles.

chad edited edge metadata.
  • fixes from testing
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 8 2016, 9:44 PM
This revision was automatically updated to reflect the committed changes.