From 840a1b186f328f04a51451da900bf9754b7c4eef Mon Sep 17 00:00:00 2001
From: Jannis Portmann <jannisp@student.ethz.ch>
Date: Mon, 20 Feb 2023 11:31:10 +0100
Subject: [PATCH 01/10] Upload and show offers

---
 pflaenzli/templates/base.html         |  2 +-
 pflaenzli/templates/offer/detail.html |  6 ++--
 pflaenzli/templates/offer/search.html | 44 +++++++++------------------
 pflaenzli/upload.py                   | 11 +++++++
 pflaenzli/urls.py                     |  1 +
 pflaenzli/views.py                    | 28 ++++++++---------
 6 files changed, 42 insertions(+), 50 deletions(-)
 create mode 100644 pflaenzli/upload.py

diff --git a/pflaenzli/templates/base.html b/pflaenzli/templates/base.html
index 257f8fa..d43af8a 100644
--- a/pflaenzli/templates/base.html
+++ b/pflaenzli/templates/base.html
@@ -48,7 +48,7 @@
                          id="navbarSupportedContent">
                         <ul class="navbar-nav mr-auto">
                             <li class="nav-item">
-                                <a class="nav-link" href="{% url 'index' %}"><i class="fa-solid fa-seedling"></i><span class="ms-2">Offers</span></a>
+                                <a class="nav-link" href="{% url 'list_offers' %}"><i class="fa-solid fa-seedling"></i><span class="ms-2">Offers</span></a>
                             </li>
                             <li class="nav-item">
                                 <a class="nav-link" href="{% url 'create_offer' %}"><i class="fa-solid fa-square-plus"></i><span class="ms-2">New Offer</span></a>
diff --git a/pflaenzli/templates/offer/detail.html b/pflaenzli/templates/offer/detail.html
index 2be77d3..6de2e12 100644
--- a/pflaenzli/templates/offer/detail.html
+++ b/pflaenzli/templates/offer/detail.html
@@ -15,8 +15,8 @@
         </div>
         <div class="show-offer-info w-50">
             <h1 class="mb-3">{{ offer.title }}</h1>
-            <div class="mb-3 d-flex">
-                <p class="pr-3">
+            <div class="mb-3 d-flex column-gap-2">
+                <p class="mr-3">
                     <i class="fas fa-user"></i>
                     {% if offer.user == user %}
                         Me
@@ -24,7 +24,7 @@
                         {{ offer.user.username }}
                     {% endif %}
                 </p>
-                <p class="pr-3">
+                <p class="mr-3">
                     <i class="fas fa-map-marker-alt"></i> {{ offer.zipcode }}
                 </p>
                 {% if distance > 0 %}
diff --git a/pflaenzli/templates/offer/search.html b/pflaenzli/templates/offer/search.html
index 7cb0c52..6cc8a7d 100644
--- a/pflaenzli/templates/offer/search.html
+++ b/pflaenzli/templates/offer/search.html
@@ -1,45 +1,29 @@
 {% extends 'base.html' %}
+{% load static %}
 {% block title %}Offers{% endblock %}
 {% block content %}
-    <div class="mb-5">
-        <a class="btn btn-pfl"
-           data-toggle="collapse"
-           href="#collapseExample"
-           role="button"
-           aria-expanded="false"
-           aria-controls="collapseExample">
-            <div class="btn btn-pfl">
-                <i class="fas fa-filter mr-3"></i>Filter<i class="fas fa-chevron-down ml-3 dropdown-collapse"></i>
-            </div>
-        </a>
-        <div class="collapse" id="collapseExample">{{ form(filter_form, {attr: {novalidate: 'novalidate' }}) }}</div>
-    </div>
     <h1>Offers</h1>
-    {% if offers|length > 0 %}
-        <div class="card-deck d-flex justify-content-around justify-content-sm-around justify-content-md-between flex-wrap">
+    {% if offers %}
+        <div class="row row-cols-1 row-cols-md-2 row-cols-xl-3 mb-3 row-gap-5">
             {% for offer in offers %}
-                <div class="mb-5">
-                    <div class="card offer h-100">
-                        <a href="{{ path('show_offer', {'urlId': offer.urlId }) }}">
-                            {% if offer.photoFilename %}
-                                <img class="card-img-top offer-img"
-                                     src="{{ asset('uploads/photos/' ~ offer.photofilename) }}"/>
+                <div class="col">
+                    <div class="card h-100 p-0">
+                        <a href="{% url 'offer_detail' offer.id %}">
+                            {% if offer.image %}
+                                <img class="card-img-top offer-img" src="{{ offer.image.url }}"/>
                             {% else %}
-                                <img class="card-img-top offer-img" src="{{ asset('placeholder.jpg') }}" />
+                                <img class="card-img-top offer-img" src="{% static 'placeholder.jpg' %}" />
                             {% endif %}
                             <div class="card-body">
-                                <h5 class="card-title">{{ offer.title }}</h5>
+                                <h5 class="h5">{{ offer.title }}</h5>
                             </div>
                         </a>
-                        <div class="card-footer offer-footer">
-                            <a class="user-link"
-                               href="{{ path('user_public', { 'urlId': offer.byuser.urlId }) }}">
-                                <p class="username">
-                                    <i class="fas fa-user mt-3"></i> {{ offer.byUser }}
-                                </p>
+                        <div class="card-footer d-flex justify-content-between">
+                            <a href="#">
+                                <i class="fas fa-user mt-3"></i> {{ offer.user.username }}
                             </a>
                             <p class="zip">
-                                <i class="fas fa-map-marker-alt mt-3"></i> {{ offer.zipCode }}
+                                <i class="fas fa-map-marker-alt mt-3"></i> {{ offer.zipcode }}
                             </p>
                         </div>
                     </div>
diff --git a/pflaenzli/upload.py b/pflaenzli/upload.py
new file mode 100644
index 0000000..b05c215
--- /dev/null
+++ b/pflaenzli/upload.py
@@ -0,0 +1,11 @@
+import os
+from pathlib import Path
+from uuid import uuid4
+from django.core.files.storage import default_storage
+
+
+def generate_unique_filename(filename):
+    file_type = Path(filename).suffix
+    new_filename = default_storage.get_available_name(uuid4())
+
+    return f'{new_filename}{file_type}'
diff --git a/pflaenzli/urls.py b/pflaenzli/urls.py
index 38d7ec9..07aa4cc 100644
--- a/pflaenzli/urls.py
+++ b/pflaenzli/urls.py
@@ -7,6 +7,7 @@ from . import views
 
 urlpatterns = [
     path("", views.index, name="index"),
+    path("offers/", views.list_offers, name="list_offers"),
     path("offer/create/", views.create_offer, name="create_offer"),
     path("offer/<int:offer_id>/", views.offer_detail, name="offer_detail"),
     path("offer/<int:offer_id>/delete", views.offer_delete, name="offer_delete"),
diff --git a/pflaenzli/views.py b/pflaenzli/views.py
index 8005aaa..82f45cb 100644
--- a/pflaenzli/views.py
+++ b/pflaenzli/views.py
@@ -1,7 +1,3 @@
-import os
-import pathlib
-import uuid
-from django.core.files.storage import default_storage
 from django.shortcuts import render, get_object_or_404, redirect
 from django.utils import timezone
 from django.contrib import messages
@@ -9,12 +5,19 @@ from django.contrib.auth.decorators import login_required
 
 from .forms import CreateOfferForm
 from .models import Offer, Wish
+from .upload import generate_unique_filename
 
 
 def index(request):
     return render(request, "app/index.html")
 
 
+def list_offers(request, filters=None):
+    offers = Offer.objects.all()
+    print(offers)
+    return render(request, "offer/search.html", {"offers": offers})
+
+
 @login_required
 def create_offer(request):
     if request.method == "POST":
@@ -22,7 +25,7 @@ def create_offer(request):
         form = CreateOfferForm(request.POST, request.FILES)
         if form.is_valid():
             offer = form.save(commit=False)
-            offer.image.name = generate_unique_filename(request.FILES['image'].name)
+            offer.image.name = generate_unique_filename(form.cleaned_data['image'].name)
             offer.user = request.user
             offer.save()
             messages.success(request, "Offer uploaded successfully!")
@@ -33,30 +36,23 @@ def create_offer(request):
     return render(request, "basic_form.html", {"form": form, "button_label": "Create"})
 
 
-@login_required
+@ login_required
 def offer_detail(request, offer_id):
     offer = get_object_or_404(Offer, id=offer_id)
 
     return render(request, "offer/detail.html", {"offer": offer, "wishes": ["Monstera", "Tradescantia"]})
 
 
-def generate_unique_filename(filename):
-    file_type = pathlib.Path(filename).suffix
-    new_filename = default_storage.get_available_name(uuid.uuid4())
-
-    return f'{new_filename}{file_type}'
-
-
-@login_required
+@ login_required
 def offer_delete(request, offer_id):
     return 0
 
 
-@login_required
+@ login_required
 def offer_edit(request, offer_id):
     return 0
 
 
-@login_required
+@ login_required
 def offer_trade(request, offer_id):
     return 0

From 27e4779816febbdf0e54263e677f9aaa36f51ec1 Mon Sep 17 00:00:00 2001
From: Jannis Portmann <jannisp@student.ethz.ch>
Date: Mon, 20 Feb 2023 11:33:21 +0100
Subject: [PATCH 02/10] Increase padding

---
 pflaenzli/templates/base.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/pflaenzli/templates/base.html b/pflaenzli/templates/base.html
index d43af8a..ea287aa 100644
--- a/pflaenzli/templates/base.html
+++ b/pflaenzli/templates/base.html
@@ -109,7 +109,7 @@
                     </div>
                 </div>
             </nav>
-            <div class="container pt-4 mb-5transparent">
+            <div class="container pt-4 mb-5">
                 <div class="container mt-5">
                     {% if messages %}
                         {% for message in messages %}
@@ -124,7 +124,7 @@
                 </div>
             </div>
         </main>
-        <footer class="mt-auto p-5 align-items-center bg-body-tertiary">
+        <footer class="mt-auto p-4 align-items-center bg-body-tertiary">
             <div class="container d-flex justify-content-around align-items-center">
                 <p>
                     <a href="#" class="me-3"><i class="fab fa-mastodon"></i></a>

From 805c3b645eabde45ebd16158134460dd491a1378 Mon Sep 17 00:00:00 2001
From: Jannis Portmann <jannisp@student.ethz.ch>
Date: Sun, 26 Mar 2023 23:07:59 +0200
Subject: [PATCH 03/10] Add admin

---
 pflaenzli/admin.py | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/pflaenzli/admin.py b/pflaenzli/admin.py
index 8c38f3f..ae34dd3 100644
--- a/pflaenzli/admin.py
+++ b/pflaenzli/admin.py
@@ -1,3 +1,12 @@
 from django.contrib import admin
+from .models import Offer, Wish
 
-# Register your models here.
+
+@admin.register(Offer)
+class OfferAdmin(admin.ModelAdmin):
+    list_display = ['id', 'title', 'user', 'zipcode']
+
+
+@admin.register(Wish)
+class WishAdmin(admin.ModelAdmin):
+    list_display = ['id', 'title', 'user']

From ca9a91c8644bc9629a6abdd1d04ddc395479dc3b Mon Sep 17 00:00:00 2001
From: Jannis Portmann <jannisp@student.ethz.ch>
Date: Fri, 31 Mar 2023 18:10:38 +0200
Subject: [PATCH 04/10] Implement edit and delete

---
 pflaenzli/models.py                   | 21 ++++++++++++++++++
 pflaenzli/templates/offer/detail.html | 31 ++++++++++++++-------------
 pflaenzli/urls.py                     |  4 ++--
 pflaenzli/views.py                    | 31 +++++++++++++++++++++++----
 4 files changed, 66 insertions(+), 21 deletions(-)

diff --git a/pflaenzli/models.py b/pflaenzli/models.py
index ce75df9..7bcb3cd 100644
--- a/pflaenzli/models.py
+++ b/pflaenzli/models.py
@@ -1,6 +1,8 @@
 from django.db import models
+from django.dispatch import receiver
 from django.contrib.auth.models import User
 from django.utils import timezone
+from django.core.files.storage import default_storage
 
 
 class Offer(models.Model):
@@ -16,3 +18,22 @@ class Wish(models.Model):
     created = models.DateTimeField(default=timezone.now)
     user = models.ForeignKey(User, on_delete=models.CASCADE)
     title = models.CharField(max_length=200)
+
+
+@receiver(models.signals.post_delete, sender=Offer)
+def auto_delete_image_on_delete(sender, instance, **kwargs):
+    """
+    Deletes image from filesystem when corresponding `Offer` object is deleted.
+    """
+    if instance.image and os.path.isfile(instance.image.path):
+        default_storage.delete(instance.image.path)
+
+
+@receiver(models.signals.pre_save, sender=Offer)
+def delete_old_image(sender, instance, **kwargs):
+    try:
+        old_image = sender.objects.get(pk=instance.pk).image
+    except sender.doesNotExist:
+        return False
+
+    default_storage.delete(old_image.path)
diff --git a/pflaenzli/templates/offer/detail.html b/pflaenzli/templates/offer/detail.html
index 6de2e12..b15400b 100644
--- a/pflaenzli/templates/offer/detail.html
+++ b/pflaenzli/templates/offer/detail.html
@@ -38,34 +38,35 @@
         </div>
     </div>
     {% if offer.user == user %}
-        <a href="{% url 'offer_edit' offer.id %}" class="btn btn-info mb-3"><i class="fas fa-pen"></i></a>
+        <a href="{% url 'offer_edit' offer.id %}" class="btn btn-info"><i class="fas fa-pen"></i></a>
+        <!-- Button trigger modal -->
         <button type="button"
-                class="btn btn-danger mb-3"
-                data-toggle="modal"
-                data-target="#exampleModal">
+                class="btn btn-danger"
+                data-bs-toggle="modal"
+                data-bs-target="#exampleModal">
             <i class="fas fa-trash-alt"></i>
         </button>
         <!-- Modal -->
         <div class="modal fade"
              id="exampleModal"
              tabindex="-1"
-             role="dialog"
              aria-labelledby="exampleModalLabel"
              aria-hidden="true">
-            <div class="modal-dialog" role="document">
+            <div class="modal-dialog">
                 <div class="modal-content">
                     <div class="modal-header">
-                        <h5 class="modal-title" id="exampleModalLabel">Warning</h5>
-                        <button type="button" class="close" data-dismiss="modal" aria-label="Close">
-                            <span aria-hidden="true">&times;</span>
-                        </button>
+                        <h1 class="modal-title fs-5" id="exampleModalLabel">Delete offer</h1>
+                        <button type="button"
+                                class="btn-close"
+                                data-bs-dismiss="modal"
+                                aria-label="Close"></button>
                     </div>
-                    <div class="modal-body">Are you sure you want to delete this offer?</div>
+                    <div class="modal-body">Do you really want to delete this offer? This can't be undone.</div>
                     <div class="modal-footer">
-                        <button type="button" class="btn btn-secondary" data-dismiss="modal">Cancel</button>
-                        <a type="button"
-                           class="btn btn-danger"
-                           href="{% url 'offer_delete' offer.id %}">Delete</a>
+                        <button type="button" class="btn btn-secondary" data-bs-dismiss="modal">Cancel</button>
+                        <a href="{% url 'offer_delete' offer.id %}"
+                           type="button"
+                           class="btn btn-danger">Delete</a>
                     </div>
                 </div>
             </div>
diff --git a/pflaenzli/urls.py b/pflaenzli/urls.py
index 07aa4cc..19c1835 100644
--- a/pflaenzli/urls.py
+++ b/pflaenzli/urls.py
@@ -10,8 +10,8 @@ urlpatterns = [
     path("offers/", views.list_offers, name="list_offers"),
     path("offer/create/", views.create_offer, name="create_offer"),
     path("offer/<int:offer_id>/", views.offer_detail, name="offer_detail"),
-    path("offer/<int:offer_id>/delete", views.offer_delete, name="offer_delete"),
-    path("offer/<int:offer_id>/edit", views.offer_edit, name="offer_edit"),
+    path("offer/<int:offer_id>/delete/", views.offer_delete, name="offer_delete"),
+    path("offer/<int:offer_id>/edit/", views.offer_edit, name="offer_edit"),
     path("offer/<int:offer_id>/trade", views.offer_trade, name="offer_trade"),
     path('accounts/login/', auth_views.LoginView.as_view(template_name='registration/login.html')),
     path('accounts/profile/', auth_views.LoginView.as_view(template_name='user/detail.html')),
diff --git a/pflaenzli/views.py b/pflaenzli/views.py
index 82f45cb..712555c 100644
--- a/pflaenzli/views.py
+++ b/pflaenzli/views.py
@@ -2,6 +2,7 @@ from django.shortcuts import render, get_object_or_404, redirect
 from django.utils import timezone
 from django.contrib import messages
 from django.contrib.auth.decorators import login_required
+from django.http import HttpResponseForbidden
 
 from .forms import CreateOfferForm
 from .models import Offer, Wish
@@ -43,14 +44,36 @@ def offer_detail(request, offer_id):
     return render(request, "offer/detail.html", {"offer": offer, "wishes": ["Monstera", "Tradescantia"]})
 
 
-@ login_required
+@login_required
 def offer_delete(request, offer_id):
-    return 0
+    offer = get_object_or_404(Offer, id=offer_id)
+    if offer.user != request.user:
+        return HttpResponseForbidden()
+
+    offer.delete()
+    messages.success(request, "Offer deleted successfully!")
+    return redirect("list_offers")
 
 
-@ login_required
+@login_required
 def offer_edit(request, offer_id):
-    return 0
+    offer = get_object_or_404(Offer, id=offer_id)
+    if offer.user != request.user:
+        return HttpResponseForbidden()
+
+    if request.method == "POST":
+        form = CreateOfferForm(request.POST, request.FILES, instance=offer)
+        if form.is_valid():
+            offer = form.save(commit=False)
+            offer.image.name = generate_unique_filename(form.cleaned_data['image'].name)
+            offer.user = request.user
+            offer.save()
+            messages.success(request, "Offer updated successfully!")
+            return redirect("offer_detail", offer.id)
+    else:
+        form = CreateOfferForm(instance=offer)
+
+    return render(request, "basic_form.html", {"form": form, "button_label": "Update"})
 
 
 @ login_required

From 18967d9178aa1686931678f6b57f12c19d554b07 Mon Sep 17 00:00:00 2001
From: Jannis Portmann <jannisp@student.ethz.ch>
Date: Fri, 31 Mar 2023 18:11:37 +0200
Subject: [PATCH 05/10] Remove print

---
 pflaenzli/views.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/pflaenzli/views.py b/pflaenzli/views.py
index 712555c..cc92d4e 100644
--- a/pflaenzli/views.py
+++ b/pflaenzli/views.py
@@ -22,7 +22,6 @@ def list_offers(request, filters=None):
 @login_required
 def create_offer(request):
     if request.method == "POST":
-        print(request.POST)
         form = CreateOfferForm(request.POST, request.FILES)
         if form.is_valid():
             offer = form.save(commit=False)

From 13e4e0fabc1f9485e71f2712784e4eb13f774aff Mon Sep 17 00:00:00 2001
From: Jannis Portmann <jannisp@student.ethz.ch>
Date: Fri, 31 Mar 2023 22:51:53 +0200
Subject: [PATCH 06/10] Fix deleting of old image

---
 pflaenzli/models.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pflaenzli/models.py b/pflaenzli/models.py
index 7bcb3cd..e0c7413 100644
--- a/pflaenzli/models.py
+++ b/pflaenzli/models.py
@@ -33,7 +33,7 @@ def auto_delete_image_on_delete(sender, instance, **kwargs):
 def delete_old_image(sender, instance, **kwargs):
     try:
         old_image = sender.objects.get(pk=instance.pk).image
-    except sender.doesNotExist:
+    except sender.DoesNotExist:
         return False
 
     default_storage.delete(old_image.path)

From 79b29d0d95e5cd1e5a357e328e5a1cead28954a1 Mon Sep 17 00:00:00 2001
From: Jannis Portmann <jannisp@student.ethz.ch>
Date: Fri, 31 Mar 2023 23:55:24 +0200
Subject: [PATCH 07/10] Add public user page

---
 pflaenzli/templates/user/public.html      | 51 ++++++++++++++++++
 pflaenzli/templates/user/public.html.twig | 63 -----------------------
 pflaenzli/urls.py                         |  3 +-
 pflaenzli/views.py                        |  8 +++
 4 files changed, 61 insertions(+), 64 deletions(-)
 create mode 100644 pflaenzli/templates/user/public.html
 delete mode 100644 pflaenzli/templates/user/public.html.twig

diff --git a/pflaenzli/templates/user/public.html b/pflaenzli/templates/user/public.html
new file mode 100644
index 0000000..f3063c7
--- /dev/null
+++ b/pflaenzli/templates/user/public.html
@@ -0,0 +1,51 @@
+{% extends "base.html" %}
+{% load static %}
+{% block title %}User {{ username }}{% endblock %}
+{% block content %}
+    <div class="mb-3">
+        <h1>{{ user.username }}'s Profile</h1>
+    </div>
+    <h2 class="h3">Wishlist</h2>
+    <div class="mb-4">
+        {% if wishes %}
+            <ul class="list-group">
+                {% for wish in wishes %}<li class="list-group-item">{{ wish.title }}</li>{% endfor %}
+            </ul>
+        {% else %}
+            <div class="alert alert-warning" role="alert">{{ user.username }} has currently no wishes!</div>
+        {% endif %}
+    </div>
+    <div class="mb-3">
+        <h2 class="h3">Offers</h2>
+    </div>
+    {% if offers %}
+        <div class="card-deck d-flex justify-content-around justify-content-sm-around justify-content-md-between flex-wrap">
+            {% for offer in offers %}
+                <div class="col">
+                    <div class="card h-100 p-0 flex-column justify-content-between">
+                        <a href="{% url 'offer_detail' offer.id %}">
+                            {% if offer.image %}
+                                <img class="card-img-top offer-img" src="{{ offer.image.url }}"/>
+                            {% else %}
+                                <img class="card-img-top offer-img" src="{% static 'placeholder.jpg' %}" />
+                            {% endif %}
+                            <div class="card-body">
+                                <h5 class="h5">{{ offer.title }}</h5>
+                            </div>
+                        </a>
+                        <div class="card-footer d-flex justify-content-between">
+                            <a href="#">
+                                <i class="fas fa-user mt-3"></i> {{ offer.user.username }}
+                            </a>
+                            <p class="zip">
+                                <i class="fas fa-map-marker-alt mt-3"></i> {{ offer.zipcode }}
+                            </p>
+                        </div>
+                    </div>
+                </div>
+            {% endfor %}
+        </div>
+    {% else %}
+        <div class="alert alert-warning" role="alert">There are currently no active offers.</div>
+    {% endif %}
+{% endblock %}
diff --git a/pflaenzli/templates/user/public.html.twig b/pflaenzli/templates/user/public.html.twig
deleted file mode 100644
index 6b65b83..0000000
--- a/pflaenzli/templates/user/public.html.twig
+++ /dev/null
@@ -1,63 +0,0 @@
-{% extends 'base.html.twig' %}
-
-{% block title %}User {{ username }}{% endblock %}
-
-{% block content %}
-    {% for message in app.flashes('success') %}
-        <div class="alert alert-success" role="alert">
-            {{ message }}
-        </div>
-    {% endfor %}
-
-    <div class="mb-3">
-        <h1>{{ username }}'s Wishlist</h1>
-    </div>
-
-    <div class="mb-4">
-    {% if wishes == [] %}
-        <div class="alert alert-warning" role="alert">
-            There are currently no wishes!
-        </div>
-    {% else %}
-        <ul class="list-group">
-        {% for wish in wishes %}
-            <li class="list-group-item"> {{ wish.title }}</li>
-        {% endfor %}
-        </ul>
-    {% endif %}
-    </div>
-    <hr>
-    <div class="mb-3">
-        <h1>{{ username }}'s Offers</h1>
-    </div>
-
-    {% if offers|length > 0 %}
-        <div class="card-deck d-flex justify-content-around justify-content-sm-around justify-content-md-between flex-wrap">
-            {% for offer in offers %}
-                <div class="mb-5">
-                    <div class="card offer h-100">
-                        <a href="{{ path('show_offer', {'urlId': offer.urlId }) }}">
-                            {% if offer.photoFilename %}
-                                <img class="card-img-top offer-img" src="{{ asset('uploads/photos/' ~ offer.photofilename) }}" />
-                            {% else %}
-                                <img class="card-img-top offer-img" src="{{ asset('placeholder.jpg') }}" />
-                            {% endif %}
-                            <div class="card-body">
-                                <h5 class="card-title">{{ offer.title }}</h5>
-                                <p class="card-text">{{ offer.description }}</p>
-                            </div>
-                        </a>
-                        <div class="card-footer offer-footer">
-                            <a class="user-link" href="{{ path('user_public', { 'urlId': offer.byuser.id }) }}">
-                                <p class="username"><i class="fas fa-user mt-3"></i> {{ offer.byUser }}</p>
-                            </a>
-                            <p class="zip"><i class="fas fa-map-marker-alt mt-3"></i> {{ offer.zipCode }}</p>
-                        </div>
-                    </div>
-                </div>
-            {% endfor %}
-        </div>
-    {% else %}
-        <div class="alert alert-warning" role="alert">There are currently no active offers.</div>
-    {% endif %}
-{% endblock %}
\ No newline at end of file
diff --git a/pflaenzli/urls.py b/pflaenzli/urls.py
index 19c1835..cb6b1e6 100644
--- a/pflaenzli/urls.py
+++ b/pflaenzli/urls.py
@@ -12,7 +12,8 @@ urlpatterns = [
     path("offer/<int:offer_id>/", views.offer_detail, name="offer_detail"),
     path("offer/<int:offer_id>/delete/", views.offer_delete, name="offer_delete"),
     path("offer/<int:offer_id>/edit/", views.offer_edit, name="offer_edit"),
-    path("offer/<int:offer_id>/trade", views.offer_trade, name="offer_trade"),
+    path("offer/<int:offer_id>/trade/", views.offer_trade, name="offer_trade"),
+    path("user/<int:user_id>", views.user_detail, name="user_detail"),
     path('accounts/login/', auth_views.LoginView.as_view(template_name='registration/login.html')),
     path('accounts/profile/', auth_views.LoginView.as_view(template_name='user/detail.html')),
     path('accounts/', include('django.contrib.auth.urls')),
diff --git a/pflaenzli/views.py b/pflaenzli/views.py
index cc92d4e..b19d033 100644
--- a/pflaenzli/views.py
+++ b/pflaenzli/views.py
@@ -78,3 +78,11 @@ def offer_edit(request, offer_id):
 @ login_required
 def offer_trade(request, offer_id):
     return 0
+
+
+def user_detail(request, user_id):
+    user = get_object_or_404(User, id=user_id)
+    offers = Offer.objects.filter(user=user_id)
+    wishes = Wish.objects.filter(user=user_id)
+
+    return render(request, "user/public.html", {"user": user, "offers": offers, "wishes": wishes})

From 9d2b219a704d2f5ffbb641d21f99d6677a26fe53 Mon Sep 17 00:00:00 2001
From: Jannis Portmann <jannisp@student.ethz.ch>
Date: Fri, 31 Mar 2023 23:56:19 +0200
Subject: [PATCH 08/10] Send request mail

---
 pflaenzli/mail.py                             | 25 +++++++++++++++++++
 pflaenzli/templates/offer/detail.html         |  2 +-
 .../templates/user/trade/offer_email.html     |  5 ++++
 .../user/trade/offer_email.html.twig          |  5 ----
 pflaenzli/views.py                            | 16 ++++++++++--
 5 files changed, 45 insertions(+), 8 deletions(-)
 create mode 100644 pflaenzli/mail.py
 create mode 100644 pflaenzli/templates/user/trade/offer_email.html
 delete mode 100644 pflaenzli/templates/user/trade/offer_email.html.twig

diff --git a/pflaenzli/mail.py b/pflaenzli/mail.py
new file mode 100644
index 0000000..d2a8504
--- /dev/null
+++ b/pflaenzli/mail.py
@@ -0,0 +1,25 @@
+from django.core.mail import EmailMessage, EmailMultiAlternatives
+from django.urls import reverse
+from django.template.loader import render_to_string
+
+
+def send_offer_email(request, offer, sender_user, recipient_user):
+    html_content = render_to_string('user/trade/offer_email.html',
+                                    {'request': request, 'offer': offer, 'sender_user': sender_user, 'recipient_user': recipient_user})
+    plain_text = get_offer_text(request, offer, sender_user, recipient_user)
+
+    message = EmailMultiAlternatives(
+        f'{sender_user.username} wants to trade',
+        plain_text,
+        'no-reply@pflaenz.li',
+        [recipient_user.email],
+        reply_to=[sender_user.email],
+    )
+
+    message.attach_alternative(html_content, 'text/html')
+
+    message.send()
+
+
+def get_offer_text(request, offer, sender_user, recipient_user):
+    return f"Hello {recipient_user.username},\n\nThe user {sender_user.username} would like to trade '{offer.title}' with you!\n\nIf you would like to trade with {sender_user.username}, just reply to this email to get in touch with them.\n\nYou can also view their offers here: {request.scheme}://{request.get_host()}{reverse('user_detail', args=[sender_user.id])}"
diff --git a/pflaenzli/templates/offer/detail.html b/pflaenzli/templates/offer/detail.html
index b15400b..00417df 100644
--- a/pflaenzli/templates/offer/detail.html
+++ b/pflaenzli/templates/offer/detail.html
@@ -83,6 +83,6 @@
                 </ul>
             {% endif %}
         </div>
-        <a class="btn btn-pfl mb-3" href="{% url 'trade' offer.id %}">Offer trade</a>
+        <a class="btn btn-pfl mb-3" href="{% url 'offer_trade' offer.id %}">Offer trade</a>
     {% endif %}
 {% endblock %}
diff --git a/pflaenzli/templates/user/trade/offer_email.html b/pflaenzli/templates/user/trade/offer_email.html
new file mode 100644
index 0000000..1042d07
--- /dev/null
+++ b/pflaenzli/templates/user/trade/offer_email.html
@@ -0,0 +1,5 @@
+<p>Hello {{ recipient_user.username }}</p>
+<h1>{{ sender_user.username }} wants to trade '{{ offer.title }}'!</h1>
+<p>Checkout their offers:</p>
+<a href="{{ request.scheme }}://{{ request.get_host }}{% url 'user_detail' sender_user.id %}">Link</a>
+<p>Just reply to get in contact witj to start trading with {{ sender_user.username }}.</p>
diff --git a/pflaenzli/templates/user/trade/offer_email.html.twig b/pflaenzli/templates/user/trade/offer_email.html.twig
deleted file mode 100644
index f23e814..0000000
--- a/pflaenzli/templates/user/trade/offer_email.html.twig
+++ /dev/null
@@ -1,5 +0,0 @@
-<h1>{{ user.username }} wants to trade!</h1>
-
-<p>Checkout {{ user.username}}'s offers:</p>
-<a href="{{ url('user_public', {'urlId': urlId}) }}">Link</a>
-<p>Reply to this email to start trading.</p>
\ No newline at end of file
diff --git a/pflaenzli/views.py b/pflaenzli/views.py
index b19d033..91d2c9d 100644
--- a/pflaenzli/views.py
+++ b/pflaenzli/views.py
@@ -2,10 +2,12 @@ from django.shortcuts import render, get_object_or_404, redirect
 from django.utils import timezone
 from django.contrib import messages
 from django.contrib.auth.decorators import login_required
+from django.contrib.auth.models import User
 from django.http import HttpResponseForbidden
 
 from .forms import CreateOfferForm
 from .models import Offer, Wish
+from .mail import send_offer_email
 from .upload import generate_unique_filename
 
 
@@ -75,9 +77,19 @@ def offer_edit(request, offer_id):
     return render(request, "basic_form.html", {"form": form, "button_label": "Update"})
 
 
-@ login_required
+@login_required
 def offer_trade(request, offer_id):
-    return 0
+    offer = get_object_or_404(Offer, id=offer_id)
+    sender = request.user
+    recipient = offer.user
+
+    if sender != recipient:
+        send_offer_email(request, offer, sender, recipient)
+        messages.success(request, f"{recipient.username} was successfully notified")
+    else:
+        messages.error(request, "You can't trade with yourself!")
+
+    return redirect("offer_detail", offer_id)
 
 
 def user_detail(request, user_id):

From 85e7440f0fbd28062434f39ef4e66acb840ae5c7 Mon Sep 17 00:00:00 2001
From: Jannis Portmann <jannisp@student.ethz.ch>
Date: Fri, 31 Mar 2023 23:56:46 +0200
Subject: [PATCH 09/10] Improve layout

---
 pflaenzli/templates/offer/detail.html | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/pflaenzli/templates/offer/detail.html b/pflaenzli/templates/offer/detail.html
index 00417df..a82121a 100644
--- a/pflaenzli/templates/offer/detail.html
+++ b/pflaenzli/templates/offer/detail.html
@@ -7,13 +7,13 @@
 {% endblock %}
 {% block content %}
     {% if offer.user == request.user %}<div class="alert alert-info" role="alert">This is your offer!</div>{% endif %}
-    <div class="show-offer-container d-flex flex-wrap">
-        <div class="show-img-container">
+    <div class="show-offer-container row gx-5">
+        <div class="show-img-container col-12 col-md-6">
             <img class="mb-3 img-fluid rounded"
                  alt="Generic placeholder image"
                  src="{{ offer.image.url }}">
         </div>
-        <div class="show-offer-info w-50">
+        <div class="show-offer-info col-12 col-md-6">
             <h1 class="mb-3">{{ offer.title }}</h1>
             <div class="mb-3 d-flex column-gap-2">
                 <p class="mr-3">
@@ -38,7 +38,7 @@
         </div>
     </div>
     {% if offer.user == user %}
-        <a href="{% url 'offer_edit' offer.id %}" class="btn btn-info"><i class="fas fa-pen"></i></a>
+        <a href="{% url 'offer_edit' offer.id %}" class="btn btn-pfl"><i class="fas fa-pen"></i></a>
         <!-- Button trigger modal -->
         <button type="button"
                 class="btn btn-danger"

From e147c67026de5ad82b83f4d1eeacb03a53d52e76 Mon Sep 17 00:00:00 2001
From: Jannis Portmann <jannisp@student.ethz.ch>
Date: Fri, 31 Mar 2023 23:57:01 +0200
Subject: [PATCH 10/10] Remove pint

---
 pflaenzli/views.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/pflaenzli/views.py b/pflaenzli/views.py
index 91d2c9d..3789ec6 100644
--- a/pflaenzli/views.py
+++ b/pflaenzli/views.py
@@ -17,7 +17,6 @@ def index(request):
 
 def list_offers(request, filters=None):
     offers = Offer.objects.all()
-    print(offers)
     return render(request, "offer/search.html", {"offers": offers})