Re: Update Commitfest requirements and README

On Sat, Jan 22, 2022 at 6:34 PM Andreas 'ads' Scherbaum <ads@pgug.de> wrote:
>
>
> Hello,
>
> While trying something out on the Commitfest, I found that the README
> and the dependencies are quite outdated. It still requires Django 1.8, as example.
> And a couple modules are missing, which are not listed in the dependencies.
>
> Attached patch takes a first shot at this problem, updates dependencies and
> README to Django 3.2 and adds all missing modules (so far).

Yeah, this looks cvorrect. Applied.


> This brings the website into a state where it at least starts.
> But then when I try to login, I run into this error:
>
> 'Settings' object has no attribute 'PGAUTH_REDIRECT'
>
> Apparently the example settings for local_settings.py are incomplete.

yeah. You need to set PGAUTH_REDIRECT and PGAUTH_KEY to have it
integrate with a local pgweb. I think the intention was at some point
to make it usable without that, and just with local accounts, but it
doesn't appear to be -- either that was never done or it has gone
broken over time. I think you can make it work locally by just
commenting out all references to the "pgcommitfest.auth" module from
urls.py?

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