public inbox for [email protected]  

help / color / mirror / Atom feed

From: Magnus Hagander <[email protected]>
To: Célestin Matte <[email protected]>
Cc: PostgreSQL WWW <[email protected]>
Subject: Re: [PATCHES] pglister: make organization name generic
Date: Sat, 15 Jan 2022 19:06:35 +0100
Message-ID: <CABUevEzBBjh7Ha9dXT4AU+0s4GmBR9sYjqXbcK_aULn=mqqY0A@mail.gmail.com> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<CABUevEw9uzwxnVqSTCgOYCYPNy2DbUW7uv7Fj3yk_ntkfpp5jg@mail.gmail.com>
	<[email protected]>

On Thu, Nov 4, 2021 at 12:21 PM Célestin Matte <[email protected]> wrote:
>
> > As a general rule, please post the patches no the thread of the description, and if it's more than one patch, in the same email. Right now the only thing I see to review is this description, that's kind of hard.
>
> Sorry, I don't understand: should I send all patches by responding to the description email (instead of replying to the previous ones, as I did here), or all patches in the same email?

All in the same email is definitely best, like you did in this one -- thanks!

> > As for reviewing the general idea, I agree with that -- makes perfect sense to do that. I did note that Alvaro spotted that it also needs to look over all the rest of the text and find similar stuff -- like the image alt="" attribute. There may be more of those?
>
> Yep, added handling of ORGANIZATION_MOTTO in the set of patches attached to this email.

Looking at 1 and 2, this just seems to me like a strange way to do it,
by declaring a new tag for it.

I think it would be cleaner with a Context Processor similar to what
we have in pgweb. Then this context processor could just return a dict
of ORGANIZATION_INFO from setitngs.py, wihch would have a key for name
and a key for motto perhaps?

As for 3 -- I don't think we want to remove the postgresql branding It
should be made optional though, perhaps by another parameter in said
dict in the settings that would just make it load from different URLs
and files?

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

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]
  Subject: Re: [PATCHES] pglister: make organization name generic
  In-Reply-To: <CABUevEzBBjh7Ha9dXT4AU+0s4GmBR9sYjqXbcK_aULn=mqqY0A@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