Re: pglister: pgarchives does not synchronize subscribers with email delivery disabled (proposed patch) - Mailing list pgsql-www

From Célestin Matte
Subject Re: pglister: pgarchives does not synchronize subscribers with email delivery disabled (proposed patch)
Date
Msg-id 3de25c28-38a9-414a-aa69-219164bd6afb@cmatte.me
Whole thread Raw
In response to Re: pglister: pgarchives does not synchronize subscribers with email delivery disabled (proposed patch)  (Magnus Hagander <magnus@hagander.net>)
List pgsql-www
> mailinglist_subscribers is used for actual mail delivery as well, isn't it? If you just change the view, it will
changeall 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
viewand 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.
Seriesof patches attached
 

Thanks for the review!
-- 
Célestin Matte
Attachment

pgsql-www by date:

Previous
From: Christoph Berg
Date:
Subject: Re: Public contributor profile pages
Next
From: Magnus Hagander
Date:
Subject: Re: Public contributor profile pages