public inbox for [email protected]  

help / color / mirror / Atom feed

From: Magnus Hagander <[email protected]>
To: Tom Lane <[email protected]>
Cc: Alvaro Herrera <[email protected]>
Cc: w^3 <[email protected]>
Subject: Re: CF app: "target version" entered twice?
Date: Thu, 28 Feb 2019 12:24:45 +0100
Message-ID: <CABUevEy5VMviknBwYC3qBZt7H28=Fkn_Fhg6Ka6ZujpukD+cUw@mail.gmail.com> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>

On Thu, Feb 28, 2019 at 12:45 AM Tom Lane <[email protected]> wrote:

> Alvaro Herrera <[email protected]> writes:
> > I just created a new patch in the commitfest app
> > (https://commitfest.postgresql.org/22/new/).  The app asked me on the
> > first page for the target version ... and then again on the second page.
> > And it didn't remember what I selected first.
>
> Yeah, I noticed the same a couple days ago.
>
> > I suppose the fix is just to remove the "target version" selectable on
> > the first page?
>
> Seems like the path of least resistance.  I'm not sure this is so
> important it has to be on the first page, anyway.
>

That wasn't even intentional :) The form had a list of "include all fields
except x,y,z" so when the field was added it was automatically added there,
and I missed it. I've now switched it to instead list the *included* fields
so this won't happen again -- and it's easier to list 2 included fields
than 12 excluded ones.

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

view thread (3+ messages)

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected]
  Subject: Re: CF app: "target version" entered twice?
  In-Reply-To: <CABUevEy5VMviknBwYC3qBZt7H28=Fkn_Fhg6Ka6ZujpukD+cUw@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox