| From: | Célestin Matte <celestin(dot)matte(at)cmatte(dot)me> |
|---|---|
| To: | Magnus Hagander <magnus(at)hagander(dot)net> |
| Cc: | PostgreSQL WWW <pgsql-www(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: pglister: pgarchives does not synchronize subscribers with email delivery disabled (proposed patch) |
| Date: | 2025-12-18 11:22:21 |
| Message-ID: | 3de25c28-38a9-414a-aa69-219164bd6afb@cmatte.me |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-www |
> mailinglist_subscribers is used for actual mail delivery as well, isn't it? If you just change the view, it will change all deliveries and basically the nomail switch to not work at all?
>
> While you're on the right track to include them, don't we also need to include the nomail column as a column in the view and then filter on that when sending email?
Correct. I added nomail in the mailinglist_subscribers view, then made just it was filtered upon in mailhandler.py. Series of patches attached
Thanks for the review!
--
Célestin Matte
| Attachment | Content-Type | Size |
|---|---|---|
| 0001-Include-nomail-users-in-API-call-results.patch | text/x-patch | 1.6 KB |
| 0002-Add-nomail-in-mailinglist_subscribers-view.patch | text/x-patch | 1.3 KB |
| 0003-mail-handler-don-t-send-email-to-users-with-nomail-s.patch | text/x-patch | 1.8 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Magnus Hagander | 2025-12-18 12:25:47 | Re: Public contributor profile pages |
| Previous Message | Christoph Berg | 2025-12-17 16:07:10 | Re: Public contributor profile pages |