Re: Dev improvements to pgweb

From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Josh Berkus <josh(at)agliodbs(dot)com>
Cc: PostgreSQL WWW <pgsql-www(at)postgresql(dot)org>
Subject: Re: Dev improvements to pgweb
Date: 2013-09-09 06:35:26
Message-ID: CABUevEy91j4X8rPVLV3-NFJnHvFHX_13wiE5DdvQ4N9ereQOpQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-www

On Sun, Sep 8, 2013 at 10:29 PM, Josh Berkus <josh(at)agliodbs(dot)com> wrote:
> Folks,
>
> First, lemme thank the web team for doing a terrific job of making it
> really easy to build a pgweb test server. I suspect that the web team
> may have cause to regret, in the future, making it easy enough that I
> can do it.
>
> Before I submit a patch covering all of the following, I'd like to make
> sure that none of the below things are intentional:
>
> 1. dev build instructions don't mention the HTML validator script
>
> 2. pyyaml is required to follow the dev build instructions, but is not
> in requirements.txt (is this because it's *only* required for dev builds?)
>
> 3. The dev instructions don't mention setting the database *user* or
> password in settings_local (probably assuming that the webserver owner
> and the db user are the same).
>
> 4. the HTML validator script has no ability to recurse or follow links;
> it only validates one page at a time.
>
> For the last item, I was thinking of writing a script which wraps the
> current HTML validator script, but either has the ability to take a set
> of URLs from a list file, or to load all files in the templates/pages
> tree below the target directory.

Nope, I wouldn't say any of those are intentional, so a patch fixing
those would definitely be accepted.

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

In response to

Browse pgsql-www by date

  From Date Subject
Next Message Magnus Hagander 2013-09-09 13:10:27 Re: Expanding "Upcoming Events" on the Homepage
Previous Message Josh Berkus 2013-09-08 20:29:08 Dev improvements to pgweb