search

regularstuff / tildeblog

A learn-to-use Django team project
BSD 3-Clause "New" or "Revised" License
2 stars 3 forks source link

Authentication #65

Open jmontoya84635 opened 2 months ago

jmontoya84635 commented 2 months ago

fixes #56

added django built in user for authentication. (need to delete sqlite and make migrations again). added 3 new views (login, logout, signup). added two html files (login.html and signup.html).

made landing page look more like sketch (login buttons up top are temporary locations)

regularstuff commented 2 months ago

I don't think this is addressing the feature of adding django-all-auth? Let me know if I'm missing the point, but:

I specifically don't want to allow users to sign up with accounts managed by the site owner, where someone might write to me and say "I lost my password," and I have to worry about validating. I want commenting by users who have signed up, but don't want any potential administrative burden. I think (don't know for sure) that django-allauth implements this.

I'm going to update #56 to explicitly say it should not use the django built in user model, unless that is a prerequisite.

Do let me know if I'm missing the point on this one.