FS#65554 - No way to disable AUR notifications when banned
Attached to Project:
AUR web interface
Opened by Alberto Salvia Novella (es20490446e) - Wednesday, 19 February 2020, 06:49 GMT
Last edited by Jonas Witschel (diabonas) - Tuesday, 02 November 2021, 10:23 GMT
Opened by Alberto Salvia Novella (es20490446e) - Wednesday, 19 February 2020, 06:49 GMT
Last edited by Jonas Witschel (diabonas) - Tuesday, 02 November 2021, 10:23 GMT
|
Details
My account is banned from the AUR, but I'm still receiving
email notifications from it with no way to disable them.
|
This task depends upon
Closed by Jonas Witschel (diabonas)
Tuesday, 02 November 2021, 10:23 GMT
Reason for closing: Implemented
Tuesday, 02 November 2021, 10:23 GMT
Reason for closing: Implemented
You meaning me. I will get my revenge.
Edit: https://lists.archlinux.org/pipermail/aur-dev/2020-July/004915.html removes the user being suspended from the PackageNotification table completely. It doesn't go beyond and notify the user atm.
Patch is a bit heavy handed, though. Looking more into this.
As represented by the conversations with multiple TUs.
(I agree there was no aurweb notification that you were suspended, but that's generally handled by the responsible TU directly emailing you.)
action_map = {
'send-resetkey': ResetKeyNotification,
'welcome': WelcomeNotification,
'comment': CommentNotification,
'update': UpdateNotification,
'flag': FlagNotification,
'adopt': AdoptNotification,
'disown': DisownNotification,
'comaintainer-add': ComaintainerAddNotification,
'comaintainer-remove': ComaintainerRemoveNotification,
'delete': DeleteNotification,
'request-open': RequestOpenNotification,
'request-close': RequestCloseNotification,
'tu-vote-reminder': TUVoteReminderNotification,
}
```
Out of these, any notifications we want to preserve while suspended?
PS: As far as I can tell in the `aurweb-notify` code, any user that's associated ends up getting notified, regardless of being suspended. The fix here is to add an extra WHERE clause to each notification we want to silence: `Users.Suspended = 0`.
Sorry for that back and forth -- was focusing too high level at first.