New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FIX: show rejected emails with unrecognized errors #5026

Merged
merged 1 commit into from Aug 4, 2017

Conversation

3 participants
@LeoMcA
Contributor

LeoMcA commented Aug 4, 2017

Although 407a236 will send rejection
messages for unrecognized errors, sometimes processing the email will
raise an error which has a blank message.

This commit:

  1. Shows rejected emails which have already been processed and contain
    a blank error in /admin/email/rejected

  2. Replaces new blank error messages with the error type

FIX: show rejected emails with unrecognized errors
Although 407a236 will send rejection
messages for unrecognized errors, sometimes processing the email will
raise an error which has a blank message.

This commit:

1. Shows rejected emails which have already been processed and contain
   a blank error in /admin/email/rejected

2. Replaces new blank error messages with the error type
@discoursebot

This comment has been minimized.

discoursebot commented Aug 4, 2017

You've signed the CLA, LeoMcA. Thank you! This pull request is ready for review.

@ZogStriP ZogStriP merged commit 99527af into discourse:master Aug 4, 2017

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment