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: Thu, 4 Nov 2021 11:29:16 +0100
Message-ID: <CABUevEw9uzwxnVqSTCgOYCYPNy2DbUW7uv7Fj3yk_ntkfpp5jg@mail.gmail.com> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>

On Tue, Nov 2, 2021 at 5:18 PM Célestin Matte <[email protected]>
wrote:

> Hello,
>
> PGLister is currently made to be run for PostgreSQL only. In an effort to
> get this software used by other organizations, this series of patches
> defines an ORGANIZATION_NAME configuration option and uses it to replace
> mentions of PostreSQL in templates.
> Additionally, new images for the banner are proposed, to replace mentions
> of PostreSQL by a sober "PGLister". Postgres' logo is, however, kept.
>

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.

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?

-- 
 Magnus Hagander
 Me: https://www.hagander.net/ <http://www.hagander.net/;
 Work: https://www.redpill-linpro.com/ <http://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: <CABUevEw9uzwxnVqSTCgOYCYPNy2DbUW7uv7Fj3yk_ntkfpp5jg@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