pagure

#1411 Allow user who wants it to be notified of their own actions
Merged by pingou. Opened by pingou.
self_notifs  into  master

Download 1411.patch

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

i hate myself for pointing out the copy/paste mistakes :tongue:

Please don't hate me for these:
1. html indentation
2. extra space: https://pagure.io/pagure/pull-request/1411#_6,15
3. copy/paste ones: https://pagure.io/pagure/pull-request/1411#_1,19 and https://pagure.io/pagure/pull-request/1411#_1,28

this can be avoided

Please don't hate me for these

I will never hate you for doing code review :)
but, I prefer if you comment on the code than give a list of links that I then have to open, separately to see what you point out :)

1 new commit added

  • Fix left over from copy/paste and code style error pointed out by @vivekanand1101

rebased

I think I got them all :)

copy/paste must have gone very very deep in your brain...so deep that you do it and yet you are unaware of it...maybe it has become an involuntary action. :tongue:

1 new commit added

  • Fix the docstring

Please don't hate me for these

I will never hate you for doing code review :)

I hope this is still valid after my doc string comment :p . :thumbsup:

Thanks for the review :)

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.