#2213 Modify 'From' Header Format
Closed by pingou. Opened by gaigelama.
gaigelama/pagure master  into  master

Download 2213.patch

In reference to Issue #2134
Modifying 'From' Header to a more preferred format.

Signed-off-by: Gaige Lama gaigelama@gmail.com

It is already in that format: https://pagure.io/fork/gaigelama/pagure/blob/master/f/pagure/lib/notify.py#_214

So there are a few things about this PR :

  • This PR only fixes a print which is done for debugging sending emails
  • It addresses a field that is already in the expected format
  • It does not address the ticket #2134 which does not rely on this piece of code but relies on the file pagure/hooks/files/git_multimail.py

I am therefore going to close this PR. If you wish to try fixing #2134 feel free to look at the file I pointed out above and open a new PR when you have it working :)

Thanks for looking into this though! :)

Pull-Request has been closed by pingou

Metadata