pagure

Clone

#4963 Include the assignee in the list of people notified on a ticket/PR
Merged 3 years ago by pingou. Opened 3 years ago by pingou.
assignee_subscriber  into  master

file modified
+4 
@@ -4957,6 +4957,10 @@ 
 

      # Add the user of the project
 

      users.add(obj.project.user.username)
 

  

+     # Add the assignee if there is one
 

+     if obj.assignee:
 

+         users.add(obj.assignee.username)
 

+ 

      # Add the regular contributors
 

      for contributor in obj.project.users:
 

          users.add(contributor.username)

We always include the assignee in the notifications (even on private
tickets), but so far we did not show them in the list of subscriber.

With this commit, this oversight is now fixed.

Fixes https://pagure.io/pagure/issue/4957

Signed-off-by: Pierre-Yves Chibon pingou@pingoured.fr

:thumbsup:

rebased onto 034022dce86007c50abba357d29e8c1bdd4d857e
3 years ago

:thumbsup:

rebased onto d4c70dc
3 years ago

Pull-Request has been merged by pingou
3 years ago
Changes Summary 1
+4 -0
file changed
pagure/lib/query.py
Powered by Pagure 5.13.3
Documentation • File an Issue • About • SSH Hostkey/Fingerprint
© Red Hat, Inc. and others.