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.

Sign up for GitHub

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

Jump to bottom

Some more e-mail notification fixes #9596

Merged
merged 2 commits into from
Jan 3, 2020
Merged

Some more e-mail notification fixes #9596

merged 2 commits into from
Jan 3, 2020

Commits on Jan 3, 2020

  1. Some more e-mail notification fixes 

    A few more small e-mail notification fixes/changes

* Style footer of notification email to be smaller
* Include text for when pull request is merged
* Don't include original body of issue or PR when merging/closing by
setting issue.Content to "" in these cases

* Set Re: prefix and meessage-ID headers based on actName instead of checking for a
comment. This fixes a bug where certain actions that didn't have a
comment were setting Message-ID instead of In-Reply-To which caused some
mail programs not to show those messages as they would have had the same
Message-ID as a previous message. Also fixes the case where a final
comment and closing message would have been displayed out of order if
you didn't have a copy of the original issue/pr cretion message.
    @mrsdizzie
    mrsdizzie committed Jan 3, 2020
    Configuration menu
    Copy the full SHA
    4dcdaef View commit details
    Browse the repository at this point in the history

  2. Update other template footers for consistency

    @mrsdizzie
    mrsdizzie committed Jan 3, 2020
    Configuration menu
    Copy the full SHA
    a3f30c1 View commit details
    Browse the repository at this point in the history