pagure

#3721 reporter and assignee headers added to notification emails
Merged by pingou. Opened by lenkaseg.
Unknown source master

Download 3721.patch

This will allow users to filter by their reported bugs and assigned bugs.

Fixes #3647

Change looks fine with two comments:

  • Could you squash all the commits into one?
  • The change to requirements.txt is already in: https://pagure.io/pagure/blob/master/f/requirements.txt#_17 so it shouldn't be needed no?

rebased onto 62bc32a93cc1e6a6174ebeed5081bbadfd8f0b5d

Done :)

Do we have tests for this portion of functionality that should be updated to check for these new headers?

Would be nice to add some, I don't think the existing tests cover this

I'll try.

rebased onto 8521c5980e77a5af1326a3b15496b7795ba338ec

I added some test.

Looks good, let's rebase and get it in :)

rebased onto 9cb3f32d450cfbacc9ad917b1a45c62e1a2908bd

Done :)

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.