Re: Deselecting "Receive Mail" on the website ml subscribe form is broken

On Fri, Oct 2, 2015 at 4:21 PM, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
wrote:

> Magnus Hagander wrote:
> > On Tue, Sep 29, 2015 at 4:14 PM, Alvaro Herrera <
> alvherre(at)2ndquadrant(dot)com>
> > wrote:
>
> > > I was looking at this when Amir first posted it, and I was wondering
> how
> > > it was possible that it ever worked in the first place. The second
> > > command should really be "set pgsql-bugs nomail" (patch below), but as
> > > you say I'm unsure that it works if the subscription hasn't been
> > > confirmed. A command that should work in either case is
> > > subscribe-set pgsql-bugs nomail
> > > subscribe-set pgsql-bugs nomail,digest
> > > but I'm not sure what's the python-ish way to express that.
> >
> > Well, if we're going to do it, let's do it right :)
> >
> > Does the attached logic seem right?
>
> Yeah, it does to me.
>
> (I assume that "lists" can in fact be a single list only.)
>
>
Applied, will go live within half an hour.

And yes, "lists" can still only be one, it's badly named.

--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/