From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Peter Geoghegan <pg(at)heroku(dot)com> |
Cc: | Peter Eisentraut <peter_e(at)gmx(dot)net>, Robert Haas <robertmhaas(at)gmail(dot)com>, Marco Nenciarini <marco(dot)nenciarini(at)2ndquadrant(dot)it>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: New CF app deployment |
Date: | 2015-02-22 20:13:58 |
Message-ID: | CABUevEwYMTJMRNES13bPRLMeXPWKfk5yZgvDHm663_YPar5VWw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sun, Feb 15, 2015 at 9:46 PM, Peter Geoghegan <pg(at)heroku(dot)com> wrote:
> On Sun, Feb 15, 2015 at 4:59 PM, Peter Eisentraut <peter_e(at)gmx(dot)net> wrote:
> > I think the old system where the patch submitter declared, this message
> > contains my patch, is the only one that will work.
>
> I tend to agree. That being said, calling out latest attachments is
> also useful (or highlighting that a particular mail has a particular
> file attached in general).
>
We can keep listing the attachment and just remove the automated check of
what *kind* of attachment it is.
Annotations-wise, I think that it would be nice to be able to modify
> an annotation, in case a typo is made (the old app supported this). I
>
I was sort of thinking it was something that happened seldom enough that
you could just delete it and remove it again. But I guess if we're
specifically thinking typos, it would make sense to add the ability to edit.
> also think that it's a waste of screen space to show "who" within the
> annotation view. Granted, the old app supported this, but I tend to
> think that if I actually cared who added a certain annotation, I'd be
> happy to drill down into history. I haven't cared so far, AFAICR.
>
Hmm. Personally, I definitely care about who made an annotation, but i
guess I could be OK with going into the history as well. Anybody else have
an opinion on the matter? I don't personally feel strongly either way.
--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2015-02-22 20:26:59 | Re: New CF app deployment |
Previous Message | Magnus Hagander | 2015-02-22 20:12:12 | Re: New CF app deployment |