From 4345134efe7075ff77dc3b9638e9cc99d759a18d Mon Sep 17 00:00:00 2001
From: Fer-Bar <linofernando2703@gmail.com>
Date: Sat, 25 May 2024 21:33:44 -0400
Subject: [PATCH 01/16] Added LOGIN_REDIRECT_URL and LOGOUT_REDIRECT_URL urls

---
 albercan_backend/settings.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/albercan_backend/settings.py b/albercan_backend/settings.py
index 6e2ad27..ae95c5b 100644
--- a/albercan_backend/settings.py
+++ b/albercan_backend/settings.py
@@ -143,3 +143,6 @@
 # https://docs.djangoproject.com/en/5.0/ref/settings/#default-auto-field
 
 DEFAULT_AUTO_FIELD = "django.db.models.BigAutoField"
+
+LOGIN_REDIRECT_URL = "/index"
+LOGOUT_REDIRECT_URL = "/account/login"

From 5af33095f0e77ff71dce1a76531b1faf99ceeb40 Mon Sep 17 00:00:00 2001
From: Fer-Bar <linofernando2703@gmail.com>
Date: Sat, 25 May 2024 21:34:33 -0400
Subject: [PATCH 02/16] Added navbar.css styles to base.html

---
 albercan_backend/templates/base.html | 1 +
 1 file changed, 1 insertion(+)

diff --git a/albercan_backend/templates/base.html b/albercan_backend/templates/base.html
index 4fdfd01..d68a3c2 100644
--- a/albercan_backend/templates/base.html
+++ b/albercan_backend/templates/base.html
@@ -20,6 +20,7 @@
     <!-- FontAwesome -->
     <script src="https://kit.fontawesome.com/31b739dec4.js" crossorigin="anonymous"></script>
     <link rel="stylesheet" href={% static "css/base.css" %}>
+    <link rel="stylesheet" href={% static "css/navbar.css" %}>
     <!-- Additional Styling -->
     {% block styles %}{% endblock %}
   </head>

From 35f4c1a99ff1bfdd8a043632e07cb384ba4e8c35 Mon Sep 17 00:00:00 2001
From: Fer-Bar <linofernando2703@gmail.com>
Date: Sat, 25 May 2024 23:27:38 -0400
Subject: [PATCH 03/16] Added logout link into navbar

---
 albercan_backend/static/css/navbar.css | 9 +++++++++
 albercan_backend/templates/navbar.html | 8 +++++++-
 2 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/albercan_backend/static/css/navbar.css b/albercan_backend/static/css/navbar.css
index 726198b..7e890b1 100644
--- a/albercan_backend/static/css/navbar.css
+++ b/albercan_backend/static/css/navbar.css
@@ -71,4 +71,13 @@ header .navbar .dropdown-menu .dropdown-item.active:hover {
 header .navbar .dropdown-menu .dropdown-item.active {
   background: #bc5148;
   color: #fff;
+}
+
+#logout-form {
+  display: inline;
+}
+#logout-form button {
+  background: none;
+  border: none;
+  cursor: pointer;
 }
\ No newline at end of file
diff --git a/albercan_backend/templates/navbar.html b/albercan_backend/templates/navbar.html
index c15d4cd..00c9eb6 100644
--- a/albercan_backend/templates/navbar.html
+++ b/albercan_backend/templates/navbar.html
@@ -1,7 +1,7 @@
 <!-- navbar.html -->
 <nav class="navbar navbar-expand-lg navbar-light bg-light">
   <div class="container">
-    <a class="navbar-brand" href="#">AlberCan</a>
+    <a class="navbar-brand" href="{% url 'pet:main_page' %}">AlberCan</a>
     <button class="navbar-toggler" type="button" data-bs-toggle="collapse"
             data-bs-target="#navbarSupportedContent" aria-controls="navbarSupportedContent"
             aria-expanded="false" aria-label="Toggle navigation">
@@ -30,6 +30,12 @@
         <li class="nav-item">
           <a class="nav-link" href="#">Contáctanos</a>
         </li>
+        <li class="nav-item">
+          <form id="logout-form" method="post" action="{% url 'people:logout' %}">
+            {% csrf_token %}
+            <button type="submit" class="nav-link">Cerrar Sesión</button>
+          </form>
+        </li>
       </ul>
     </div>
   </div>

From 4e27355411e5809a49fa0ccf9fa502a62cbf8c28 Mon Sep 17 00:00:00 2001
From: Fer-Bar <linofernando2703@gmail.com>
Date: Sun, 26 May 2024 00:05:16 -0400
Subject: [PATCH 04/16] Moved creation and deactivation logic to signals.py

---
 people/models.py | 23 -----------------------
 1 file changed, 23 deletions(-)

diff --git a/people/models.py b/people/models.py
index 886eca9..af3341a 100644
--- a/people/models.py
+++ b/people/models.py
@@ -154,29 +154,6 @@ def picture_url(self):
         if self.picture and hasattr(self.picture, "url"):
             return self.picture.url
 
-    def create_user(self):
-        if self.user_id is not None:
-            return
-        self.user, _ = User.objects.get_or_create(
-            username=self.personal_email,
-            defaults={
-                "email": self.personal_email,
-                "first_name": self.first_name,
-                "last_name": self.last_name,
-                "is_staff": True
-            }
-        )
-
-    def deactivate_user(self):
-        if not self.user_id:
-            return
-        self.user.is_staff = False
-        self.user.is_active = False
-        self.user.save()
-
-    def save(self, *args, **kwargs):
-        super().save(*args, **kwargs)
-
     def __str__(self):
         return f"{self.last_name}, {self.first_name}"
 

From a3a839c3344e37f821702c29c6302c9a2a4619d4 Mon Sep 17 00:00:00 2001
From: Fer-Bar <linofernando2703@gmail.com>
Date: Sun, 26 May 2024 00:18:42 -0400
Subject: [PATCH 05/16] Added pet as app_name to urls.py

---
 pet/urls.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/pet/urls.py b/pet/urls.py
index ca572c0..cbd96a3 100644
--- a/pet/urls.py
+++ b/pet/urls.py
@@ -2,6 +2,7 @@
 
 from pet.views import main_page
 
+app_name = 'pet'
 urlpatterns = [
     path("index/", main_page, name="main_page"),
 ]

From 1f67a4b0e7212b7ce6fe09627e66d0e4513741fa Mon Sep 17 00:00:00 2001
From: Fer-Bar <linofernando2703@gmail.com>
Date: Sun, 26 May 2024 00:33:06 -0400
Subject: [PATCH 06/16] Created new application for authentication

---
 people/tests.py => authentication/__init__.py | 0
 authentication/migrations/__init__.py         | 0
 2 files changed, 0 insertions(+), 0 deletions(-)
 rename people/tests.py => authentication/__init__.py (100%)
 create mode 100644 authentication/migrations/__init__.py

diff --git a/people/tests.py b/authentication/__init__.py
similarity index 100%
rename from people/tests.py
rename to authentication/__init__.py
diff --git a/authentication/migrations/__init__.py b/authentication/migrations/__init__.py
new file mode 100644
index 0000000..e69de29

From 24f77d0574c40d58a5cccf6c55c5d5bcda277907 Mon Sep 17 00:00:00 2001
From: Fer-Bar <linofernando2703@gmail.com>
Date: Sun, 26 May 2024 00:33:16 -0400
Subject: [PATCH 07/16] Created new application for authentication

---
 authentication/apps.py | 6 ++++++
 1 file changed, 6 insertions(+)
 create mode 100644 authentication/apps.py

diff --git a/authentication/apps.py b/authentication/apps.py
new file mode 100644
index 0000000..8bab8df
--- /dev/null
+++ b/authentication/apps.py
@@ -0,0 +1,6 @@
+from django.apps import AppConfig
+
+
+class AuthenticationConfig(AppConfig):
+    default_auto_field = 'django.db.models.BigAutoField'
+    name = 'authentication'

From edc13923ab00efb72e38119092c6320d73d1dab1 Mon Sep 17 00:00:00 2001
From: Fer-Bar <linofernando2703@gmail.com>
Date: Sun, 26 May 2024 00:34:43 -0400
Subject: [PATCH 08/16] Added UserLoginForm, who inherited from
 AuthenticationForm

---
 authentication/forms.py | 5 +++++
 1 file changed, 5 insertions(+)
 create mode 100644 authentication/forms.py

diff --git a/authentication/forms.py b/authentication/forms.py
new file mode 100644
index 0000000..56c6f65
--- /dev/null
+++ b/authentication/forms.py
@@ -0,0 +1,5 @@
+from django.contrib.auth.forms import AuthenticationForm
+
+
+class UserLoginForm(AuthenticationForm):
+    pass

From d62682c98f6c875adff01de94c4e6588bb708710 Mon Sep 17 00:00:00 2001
From: Fer-Bar <linofernando2703@gmail.com>
Date: Sun, 26 May 2024 00:37:20 -0400
Subject: [PATCH 09/16] Added login and logout views and urls

---
 authentication/urls.py | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)
 create mode 100644 authentication/urls.py

diff --git a/authentication/urls.py b/authentication/urls.py
new file mode 100644
index 0000000..1fdcbe1
--- /dev/null
+++ b/authentication/urls.py
@@ -0,0 +1,20 @@
+from django.contrib.auth.views import LoginView, LogoutView
+from django.urls import path
+
+from authentication.forms import UserLoginForm
+
+app_name = "authentication"
+urlpatterns = [
+    path(
+        "login/",
+        LoginView.as_view(
+            template_name="authentication/login_page.html", form_class=UserLoginForm
+        ),
+        name="login",
+    ),
+    path(
+        "logout/",
+        LogoutView.as_view(),
+        name="logout",
+    ),
+]

From a079f8aed5723901a013d72a77cf24b2658787d6 Mon Sep 17 00:00:00 2001
From: Fer-Bar <linofernando2703@gmail.com>
Date: Sun, 26 May 2024 00:38:19 -0400
Subject: [PATCH 10/16] Updated LOGOUT_REDIRECT_URL with the new url, and added
 a comma in INSTALLED_APPS

---
 albercan_backend/settings.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/albercan_backend/settings.py b/albercan_backend/settings.py
index ae95c5b..6568e66 100644
--- a/albercan_backend/settings.py
+++ b/albercan_backend/settings.py
@@ -41,7 +41,8 @@
     "django.contrib.staticfiles",
     "people",
     "pet",
-    "structure"
+    "structure",
+    "authentication"
 ]
 
 MIDDLEWARE = [
@@ -145,4 +146,4 @@
 DEFAULT_AUTO_FIELD = "django.db.models.BigAutoField"
 
 LOGIN_REDIRECT_URL = "/index"
-LOGOUT_REDIRECT_URL = "/account/login"
+LOGOUT_REDIRECT_URL = "/auth/login"

From ba12955db87eb77cd3c2f62b47ab08cc3cb33067 Mon Sep 17 00:00:00 2001
From: Fer-Bar <linofernando2703@gmail.com>
Date: Sun, 26 May 2024 00:39:03 -0400
Subject: [PATCH 11/16] Added login_page template

---
 .../templates/authentication/login_page.html  | 30 +++++++++++++++++++
 1 file changed, 30 insertions(+)
 create mode 100644 authentication/templates/authentication/login_page.html

diff --git a/authentication/templates/authentication/login_page.html b/authentication/templates/authentication/login_page.html
new file mode 100644
index 0000000..6960b7c
--- /dev/null
+++ b/authentication/templates/authentication/login_page.html
@@ -0,0 +1,30 @@
+{% extends "base.html" %}
+{% load static %}
+
+{% block title %}
+    Página de Inicio de Sesión
+{% endblock %}
+
+{% block styles %}
+{% endblock %}
+
+{% block content %}
+{% if messages %}
+  <ul class="messages">
+    {% for message in messages %}
+      <li{% if message.tags %} class="{{ message.tags }}"{% endif %}>{{ message }}</li>
+    {% endfor %}
+  </ul>
+{% endif %}
+
+<form action="" method="post">
+  {% csrf_token %}
+  {{ form.as_p }}
+  <input type="submit" value="Submit">
+  <p><a href="#">Don't have an account?</a></p>
+</form>
+
+{% endblock %}
+
+{% block scripts %}
+{% endblock %}
\ No newline at end of file

From 217118368c045f41832c62147d82799ceb2ad216 Mon Sep 17 00:00:00 2001
From: Fer-Bar <linofernando2703@gmail.com>
Date: Sun, 26 May 2024 00:39:23 -0400
Subject: [PATCH 12/16] Updated logout url to a new one

---
 albercan_backend/templates/navbar.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/albercan_backend/templates/navbar.html b/albercan_backend/templates/navbar.html
index 00c9eb6..9805d30 100644
--- a/albercan_backend/templates/navbar.html
+++ b/albercan_backend/templates/navbar.html
@@ -31,7 +31,7 @@
           <a class="nav-link" href="#">Contáctanos</a>
         </li>
         <li class="nav-item">
-          <form id="logout-form" method="post" action="{% url 'people:logout' %}">
+          <form id="logout-form" method="post" action="{% url 'authentication:logout' %}">
             {% csrf_token %}
             <button type="submit" class="nav-link">Cerrar Sesión</button>
           </form>

From 0ef9b2a6899fc01f9583730f515eef8881b07c18 Mon Sep 17 00:00:00 2001
From: Fer-Bar <linofernando2703@gmail.com>
Date: Sun, 26 May 2024 00:40:23 -0400
Subject: [PATCH 13/16] Added auth and people urls to urlpatterns

---
 albercan_backend/urls.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/albercan_backend/urls.py b/albercan_backend/urls.py
index faa63e3..a8a8e25 100644
--- a/albercan_backend/urls.py
+++ b/albercan_backend/urls.py
@@ -4,6 +4,8 @@
 from django.urls import include, path
 
 urlpatterns = [
-    path('', include('pet.urls')),
+    path('', include('pet.urls', namespace='pet')),
+    path('', include('people.urls', namespace='people')),
+    path('auth/', include('authentication.urls', namespace='authentication')),
     path("admin/", admin.site.urls),
 ] + static(settings.MEDIA_URL, document_root=settings.MEDIA_ROOT)

From d5a6ac1dd1d6f33be12449af270ca38a0d148bd4 Mon Sep 17 00:00:00 2001
From: Fer-Bar <linofernando2703@gmail.com>
Date: Sun, 26 May 2024 00:41:08 -0400
Subject: [PATCH 14/16] Added PersonListView to show all persons who are staff

---
 people/urls.py  |  8 ++++++++
 people/views.py | 12 ++++++++++++
 2 files changed, 20 insertions(+)
 create mode 100644 people/urls.py

diff --git a/people/urls.py b/people/urls.py
new file mode 100644
index 0000000..bc241d9
--- /dev/null
+++ b/people/urls.py
@@ -0,0 +1,8 @@
+from django.urls import path
+
+from people.views import PersonListView
+
+app_name = "people"
+urlpatterns = [
+    path("members/", PersonListView.as_view(), name="member-list"),
+]
diff --git a/people/views.py b/people/views.py
index e69de29..9cc8ed8 100644
--- a/people/views.py
+++ b/people/views.py
@@ -0,0 +1,12 @@
+from django.utils.decorators import method_decorator
+from django.views.decorators.http import require_http_methods
+from django.views.generic import ListView
+
+from people.models import Person
+
+
+@method_decorator(require_http_methods(["GET"]), name='dispatch')
+class PersonListView(ListView):
+    queryset = Person.objects.filter(user__is_staff=True)
+    context_object_name = "people"
+    template_name = "people/members.html"

From 6c4d5bc5b8396df259359c48b1018b8602ce6ac4 Mon Sep 17 00:00:00 2001
From: Fer-Bar <linofernando2703@gmail.com>
Date: Sun, 26 May 2024 00:41:26 -0400
Subject: [PATCH 15/16] Added members template to show all persons who are
 staff

---
 people/templates/people/members.html | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)
 create mode 100644 people/templates/people/members.html

diff --git a/people/templates/people/members.html b/people/templates/people/members.html
new file mode 100644
index 0000000..01f6be0
--- /dev/null
+++ b/people/templates/people/members.html
@@ -0,0 +1,24 @@
+{% extends "base.html" %}
+{% load static %}
+
+{% block title %}
+    Nuestros Miembros
+{% endblock %}
+
+{% block styles %}
+{% endblock %}
+
+{% block content %}
+
+<ul>
+  {% for person in people %}
+    <li>{{ person }}</li>
+  {% endfor %}
+</ul>
+
+
+
+{% endblock %}
+
+{% block scripts %}
+{% endblock %}
\ No newline at end of file

From 6bb60497d03f076eeb1c9cdf48610a7f7311ee31 Mon Sep 17 00:00:00 2001
From: Fer-Bar <linofernando2703@gmail.com>
Date: Sun, 26 May 2024 00:52:18 -0400
Subject: [PATCH 16/16] Updated reverse links present in tests

---
 pet/tests.py | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/pet/tests.py b/pet/tests.py
index 69a89d9..be557b0 100644
--- a/pet/tests.py
+++ b/pet/tests.py
@@ -24,25 +24,25 @@ def setUp(self):
         )
 
     def test_main_page_status_code(self):
-        response = self.client.get(reverse("main_page"))
+        response = self.client.get(reverse("pet:main_page"))
         self.assertEqual(response.status_code, 200)
 
     def test_main_page_template_used(self):
-        response = self.client.get(reverse("main_page"))
+        response = self.client.get(reverse("pet:main_page"))
         self.assertTemplateUsed(response, "pet/main_page.html")
 
     def test_main_page_contains_correct_html(self):
-        response = self.client.get(reverse("main_page"))
+        response = self.client.get(reverse("pet:main_page"))
         self.assertContains(response, "¡Haz un cambio hoy!")
         self.assertContains(response, "Pet1")
         self.assertContains(response, "Pet2")
 
     def test_main_page_does_not_contain_incorrect_html(self):
-        response = self.client.get(reverse("main_page"))
+        response = self.client.get(reverse("pet:main_page"))
         self.assertNotContains(response, "Hello, world!")
 
     def test_main_page_context(self):
-        response = self.client.get(reverse("main_page"))
+        response = self.client.get(reverse("pet:main_page"))
         self.assertTrue("pets" in response.context)
         self.assertTrue("headlines" in response.context)
         self.assertEqual(len(response.context["pets"]), 2)