Re: [PATCHES] pglister: make organization name generic

From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Célestin Matte <celestin(dot)matte(at)cmatte(dot)me>
Cc: PostgreSQL WWW <pgsql-www(at)lists(dot)postgresql(dot)org>
Subject: Re: [PATCHES] pglister: make organization name generic
Date: 2022-01-15 18:06:35
Message-ID: CABUevEzBBjh7Ha9dXT4AU+0s4GmBR9sYjqXbcK_aULn=mqqY0A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-www

On Thu, Nov 4, 2021 at 12:21 PM Célestin Matte <celestin(dot)matte(at)cmatte(dot)me> 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/

In response to

Responses

Browse pgsql-www by date

  From Date Subject
Next Message Magnus Hagander 2022-01-16 16:34:24 Re: missing ML messages
Previous Message Magnus Hagander 2022-01-15 17:54:20 Re: [PATCH 1/2] pgarchives: Add missing authentication backend to settings.py