Add username/password authentication

[krismuniz]

Video

https://user-images.githubusercontent.com/860507/133983101-629aa41c-a1ad-463c-86ff-cbcbe683ca50.mp4

Added

• SignUpView & form
• Log in view from django.contrib.auth
• Add simple example ProfileView to showcase protected url

Updated

• Config, login url
• Config, login redirect url
• Config, logout redirect url

Tasks

• [x] User must be able to sign up using email and password
• [x] User must be able to log in using email and password
• [x] Authenticated user must be able to see protected page
• [x] Unanthenticated used must not be allowed to see a protected page
• [x] Authenticated user must not be able to see login/signup pages

Additional Comments

Please offer as much feedback as possible as I'm not very familiar with Django and related good practices 🙏🏼 ¡gracias!

Also my first time using Tailwind with something other than React or Vue. There's a lot of repetition that I am not used to, not sure how to "componentize" UI for Django apps. Let me know if you know of a better way!!

Closes #4

[gcollazo]

Voy a review esto pronto.

En el tema de la repetición y tailwind estoy de acuerdo. El sistema de templates de django tiene la posibilidad de include otros templates que podría servir como una forma simple de crear componentes. En otros proyectos (en node) he hecho algo similar con bastantes buenos resultados.

Una idea sería hacer algo así:

{% include "components/test_input.html" with prop="value" only %}

Usando este syntax solo la variable prop con valor value estaría disponible en el template test_input.html de esta forma tendrías algo similar a un componente.

Me gustaría ver que tiene que decir @rnegron que creo usa tailwind en otros proyectos de Django.

[rnegron]

De acuerdo con @gcollazo en que una buena manera built-in de manejar la repetición es creando pequeños "template components" que luego se pueden include en otros componentes.

Aqui hay más info sobre esto del include tag (y el extends que está relacionado para propósitos de eliminar duplicidad)

• https://docs.djangoproject.com/en/3.2/ref/templates/builtins/#include
• https://docs.djangoproject.com/en/3.2/ref/templates/builtins/#extends

[rnegron]

@krismuniz Tal vez te sea útil hacerle merge o rebase a este branch encima del main branch que ahora tiene varios modelos y otras cosas.

Me gusta que usaste en built-in Django auth system! Para el scope de este proyecto no hace falta nada más complicado. Pero por si te interesa ver algunos third-party packages que implementan auth, aquí algunos populares:

• https://github.com/pennersr/django-allauth
• https://github.com/python-social-auth/social-app-django

[krismuniz]

@rnegron Yup, makes sense. Sorry que no le he dado follow up a esto. He estado de vacaciones los pasados días. Next week I'll be back on this

[rnegron]

@krismuniz No problem! No sabia, disfruta las vacaciones! 🙌🏻

[krismuniz]

@rnegron @gcollazo I rebased the branch from the current state of main works as expected.

I also exteacted the field group template into its own file but I stopped myself from abstacting anything else as we are early into building the UI. It will make more sense to abstract this once we have decided the design specs/system for the rest of the product's UI.

[gcollazo]

@rnegron ¿Algún comment?

[krismuniz]

💥