Skip to content
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 #78: Email check settings #79

Merged
merged 5 commits into from
May 2, 2016
Merged

Conversation

mwarkentin
Copy link
Owner

@mwarkentin mwarkentin commented Apr 27, 2016

  • Add WATCHMAN_EMAIL_HEADERS setting
  • Add WATCHMAN_EMAIL_RECIPIENTS setting
  • Merge headers before sending email

TODO

  • Tests
  • Update docs

@mwarkentin
Copy link
Owner Author

By analyzing the blame information on this pull request, we identified @benwebber, @kezabelle and @none to be potential reviewers.

* Add `WATCHMAN_EMAIL_HEADERS` setting
* Merge headers before sending email
@dlanger
Copy link

dlanger commented Apr 27, 2016

👍

@coveralls
Copy link

coveralls commented Apr 27, 2016

Coverage Status

Coverage decreased (-0.3%) to 82.906% when pulling 194fc59 on 78-custom-email-headers into d8bc755 on master.

@coveralls
Copy link

coveralls commented May 2, 2016

Coverage Status

Coverage increased (+1.9%) to 85.043% when pulling 53625f0 on 78-custom-email-headers into d8bc755 on master.

@coveralls
Copy link

coveralls commented May 2, 2016

Coverage Status

Coverage increased (+1.9%) to 85.106% when pulling 7cc230c on 78-custom-email-headers into d8bc755 on master.

@dlanger
Copy link

dlanger commented May 2, 2016

🚀

@coveralls
Copy link

coveralls commented May 2, 2016

Coverage Status

Coverage increased (+1.9%) to 85.106% when pulling ad927f6 on 78-custom-email-headers into d8bc755 on master.

@coveralls
Copy link

coveralls commented May 2, 2016

Coverage Status

Coverage increased (+1.9%) to 85.106% when pulling fdc2b0f on 78-custom-email-headers into d8bc755 on master.

@coveralls
Copy link

coveralls commented May 2, 2016

Coverage Status

Coverage increased (+1.9%) to 85.106% when pulling fdc2b0f on 78-custom-email-headers into d8bc755 on master.

@mwarkentin mwarkentin changed the title Fix #78: Add support for custom email headers Fix #78: Email check settings May 2, 2016
@mwarkentin mwarkentin merged commit 7f2e9fa into master May 2, 2016
@mwarkentin mwarkentin deleted the 78-custom-email-headers branch May 2, 2016 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants