From dddb431d33ab7affac53db88eab499a140d4508e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20M=C3=BCller?= <mail@devmount.de>
Date: Thu, 18 Apr 2024 11:07:46 +0200
Subject: [PATCH 1/5] =?UTF-8?q?=F0=9F=94=A8=20fix=20case=20sensitive=20fil?=
 =?UTF-8?q?ename=20calls=20=F0=9F=99=88?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 frontend/src/elements/PrimaryButton.vue   | 2 +-
 frontend/src/elements/SecondaryButton.vue | 2 +-
 frontend/src/elements/TextButton.vue      | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/frontend/src/elements/PrimaryButton.vue b/frontend/src/elements/PrimaryButton.vue
index 54cbe2ba9..ec950cfc8 100644
--- a/frontend/src/elements/PrimaryButton.vue
+++ b/frontend/src/elements/PrimaryButton.vue
@@ -36,7 +36,7 @@
 <script setup>
 import { ref } from 'vue';
 import { useI18n } from 'vue-i18n';
-import ToolTip from '@/elements/Tooltip';
+import ToolTip from '@/elements/ToolTip';
 
 // icons
 import { IconClipboardCheck, IconCopy } from '@tabler/icons-vue';
diff --git a/frontend/src/elements/SecondaryButton.vue b/frontend/src/elements/SecondaryButton.vue
index 67afb1e9b..abfd3a778 100644
--- a/frontend/src/elements/SecondaryButton.vue
+++ b/frontend/src/elements/SecondaryButton.vue
@@ -36,7 +36,7 @@
 <script setup>
 import { ref } from 'vue';
 import { useI18n } from 'vue-i18n';
-import ToolTip from '@/elements/Tooltip';
+import ToolTip from '@/elements/ToolTip';
 
 // icons
 import { IconClipboardCheck, IconCopy } from '@tabler/icons-vue';
diff --git a/frontend/src/elements/TextButton.vue b/frontend/src/elements/TextButton.vue
index f892fe77e..7e7692717 100644
--- a/frontend/src/elements/TextButton.vue
+++ b/frontend/src/elements/TextButton.vue
@@ -25,7 +25,7 @@
 <script setup>
 import { ref } from 'vue';
 import { useI18n } from 'vue-i18n';
-import ToolTip from '@/elements/Tooltip';
+import ToolTip from '@/elements/ToolTip';
 
 // icons
 import { IconCopy, IconClipboardCheck } from '@tabler/icons-vue';

From d39ed4f703dbb5968d5085097215017c52ba564b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20M=C3=BCller?= <mail@devmount.de>
Date: Thu, 18 Apr 2024 11:08:36 +0200
Subject: [PATCH 2/5] =?UTF-8?q?=E2=9E=95=20show=20timezone=20for=20attende?=
 =?UTF-8?q?e=20in=20confirmation=20email?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 backend/src/appointment/database/models.py    |  1 +
 backend/src/appointment/database/schemas.py   |  1 +
 ...0823-89e1197d980d_add_attendee_timezone.py | 24 +++++++++++++++++++
 backend/src/appointment/routes/schedule.py    |  9 +++++--
 frontend/src/components/BookingModal.vue      | 10 ++++----
 5 files changed, 37 insertions(+), 8 deletions(-)
 create mode 100644 backend/src/appointment/migrations/versions/2024_04_18_0823-89e1197d980d_add_attendee_timezone.py

diff --git a/backend/src/appointment/database/models.py b/backend/src/appointment/database/models.py
index d8812167b..fb7f184e0 100644
--- a/backend/src/appointment/database/models.py
+++ b/backend/src/appointment/database/models.py
@@ -170,6 +170,7 @@ class Attendee(Base):
     id = Column(Integer, primary_key=True, index=True)
     email = Column(StringEncryptedType(String, secret, AesEngine, "pkcs5", length=255), index=True)
     name = Column(StringEncryptedType(String, secret, AesEngine, "pkcs5", length=255), index=True)
+    timezone = Column(String(255), index=True)
 
     slots = relationship("Slot", cascade="all,delete", back_populates="attendee")
 
diff --git a/backend/src/appointment/database/schemas.py b/backend/src/appointment/database/schemas.py
index a30bc4371..d1ad20896 100644
--- a/backend/src/appointment/database/schemas.py
+++ b/backend/src/appointment/database/schemas.py
@@ -30,6 +30,7 @@
 class AttendeeBase(BaseModel):
     email: str
     name: str | None = None
+    timezone: str | None = None
 
 
 class Attendee(AttendeeBase):
diff --git a/backend/src/appointment/migrations/versions/2024_04_18_0823-89e1197d980d_add_attendee_timezone.py b/backend/src/appointment/migrations/versions/2024_04_18_0823-89e1197d980d_add_attendee_timezone.py
new file mode 100644
index 000000000..5d70c2a18
--- /dev/null
+++ b/backend/src/appointment/migrations/versions/2024_04_18_0823-89e1197d980d_add_attendee_timezone.py
@@ -0,0 +1,24 @@
+"""add attendee timezone
+
+Revision ID: 89e1197d980d
+Revises: fadd0d1ef438
+Create Date: 2024-04-18 08:23:55.660065
+
+"""
+from alembic import op
+import sqlalchemy as sa
+
+
+# revision identifiers, used by Alembic.
+revision = '89e1197d980d'
+down_revision = 'fadd0d1ef438'
+branch_labels = None
+depends_on = None
+
+
+def upgrade() -> None:
+    op.add_column('attendees', sa.Column('timezone', sa.String(255), index=True))
+
+
+def downgrade() -> None:
+    op.drop_column('attendees', 'timezone')
diff --git a/backend/src/appointment/routes/schedule.py b/backend/src/appointment/routes/schedule.py
index f5e545ef4..ef38c2a43 100644
--- a/backend/src/appointment/routes/schedule.py
+++ b/backend/src/appointment/routes/schedule.py
@@ -220,7 +220,12 @@ def request_schedule_availability_slot(
     # human readable date in subscribers timezone
     # TODO: handle locale date representation
     date = slot.start.replace(tzinfo=timezone.utc).astimezone(ZoneInfo(subscriber.timezone)).strftime("%c")
-    date = f"{date}, {slot.duration} minutes"
+    date = f"{date}, {slot.duration} minutes ({subscriber.timezone})"
+
+    # human readable date in attendee timezone
+    # TODO: handle locale date representation
+    attendee_date = slot.start.replace(tzinfo=timezone.utc).astimezone(ZoneInfo(slot.attendee.timezone)).strftime("%c")
+    attendee_date = f"{attendee_date}, {slot.duration} minutes ({slot.attendee.timezone})"
 
     # Create a pending appointment
     attendee_name = slot.attendee.name if slot.attendee.name is not None else slot.attendee.email
@@ -246,7 +251,7 @@ def request_schedule_availability_slot(
     background_tasks.add_task(send_confirmation_email, url=url, attendee=attendee, date=date, to=subscriber.email)
 
     # Sending pending email to attendee
-    background_tasks.add_task(send_pending_email, owner=subscriber, date=date, to=slot.attendee.email)
+    background_tasks.add_task(send_pending_email, owner=subscriber, date=attendee_date, to=slot.attendee.email)
 
     # Mini version of slot, so we can grab the newly created slot id for tests
     return schemas.SlotOut(
diff --git a/frontend/src/components/BookingModal.vue b/frontend/src/components/BookingModal.vue
index 946ce240e..a3121ffca 100644
--- a/frontend/src/components/BookingModal.vue
+++ b/frontend/src/components/BookingModal.vue
@@ -79,9 +79,7 @@
 </template>
 
 <script setup>
-import {
-  inject, computed, reactive, ref, onMounted,
-} from 'vue';
+import { inject, computed, reactive, ref, onMounted } from 'vue';
 import { timeFormat } from '@/utils';
 import { useI18n } from 'vue-i18n';
 import { useRoute } from 'vue-router';
@@ -111,15 +109,14 @@ const props = defineProps({
 
 // Store
 const bookingModalStore = useBookingModalStore();
-const {
-  open, state, stateData, isLoading, hasErrors, isFinished, isEditable,
-} = storeToRefs(bookingModalStore);
+const { open, state, stateData, isLoading, hasErrors, isFinished, isEditable } = storeToRefs(bookingModalStore);
 
 // Refs
 
 const attendee = reactive({
   name: '',
   email: '',
+  timezone: dj.tz.guess(),
 });
 
 const bookingForm = ref();
@@ -145,6 +142,7 @@ onMounted(() => {
   if (user.exists()) {
     attendee.name = user.data.name;
     attendee.email = user.data.email;
+    attendee.timezone = user.data.timezone;
   }
 });
 </script>

From b004c10ef26c87326e7315b10c5fa814b9278ac3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20M=C3=BCller?= <mail@devmount.de>
Date: Thu, 18 Apr 2024 11:24:07 +0200
Subject: [PATCH 3/5] =?UTF-8?q?=F0=9F=94=A8=20extend=20schedule=20test=20w?=
 =?UTF-8?q?ith=20attendee=20timezone?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 backend/test/integration/test_schedule.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/backend/test/integration/test_schedule.py b/backend/test/integration/test_schedule.py
index 469107eee..2e750d267 100644
--- a/backend/test/integration/test_schedule.py
+++ b/backend/test/integration/test_schedule.py
@@ -462,7 +462,8 @@ def bust_cached_events(self, all_calendars = False):
             ),
             attendee=schemas.AttendeeBase(
                 email='hello@example.org',
-                name='Greg'
+                name='Greg',
+                timezone='Europe/Berlin'
             )
         ).model_dump(mode='json')
 

From f6da10741a51b2463c13726bca30bc28484de761 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20M=C3=BCller?= <mail@devmount.de>
Date: Thu, 18 Apr 2024 17:30:28 +0200
Subject: [PATCH 4/5] =?UTF-8?q?=F0=9F=94=A8=20Make=20attendee=20timezone?=
 =?UTF-8?q?=20mandatory?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 backend/src/appointment/database/schemas.py | 2 +-
 backend/src/appointment/routes/schedule.py  | 6 +++++-
 backend/test/unit/test_mailer.py            | 2 +-
 3 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/backend/src/appointment/database/schemas.py b/backend/src/appointment/database/schemas.py
index d1ad20896..9885e0b51 100644
--- a/backend/src/appointment/database/schemas.py
+++ b/backend/src/appointment/database/schemas.py
@@ -30,7 +30,7 @@
 class AttendeeBase(BaseModel):
     email: str
     name: str | None = None
-    timezone: str | None = None
+    timezone: str
 
 
 class Attendee(AttendeeBase):
diff --git a/backend/src/appointment/routes/schedule.py b/backend/src/appointment/routes/schedule.py
index ef38c2a43..b07b40fe3 100644
--- a/backend/src/appointment/routes/schedule.py
+++ b/backend/src/appointment/routes/schedule.py
@@ -421,5 +421,9 @@ def decide_on_schedule_availability_slot(
 
     return schemas.AvailabilitySlotAttendee(
         slot=schemas.SlotBase(start=slot.start, duration=slot.duration),
-        attendee=schemas.AttendeeBase(email=slot.attendee.email, name=slot.attendee.name)
+        attendee=schemas.AttendeeBase(
+            email=slot.attendee.email,
+            name=slot.attendee.name,
+            timezone=slot.attendee.timezone
+        )
     )
diff --git a/backend/test/unit/test_mailer.py b/backend/test/unit/test_mailer.py
index 25b75c3b2..edfb3a81c 100644
--- a/backend/test/unit/test_mailer.py
+++ b/backend/test/unit/test_mailer.py
@@ -17,7 +17,7 @@ def test_confirm(self, faker, with_l10n):
         deny_url = 'https://example.org/no'
         fake_email = 'to@example.org'
         now = datetime.datetime.now()
-        attendee = schemas.AttendeeBase(email=faker.email(), name=faker.name())
+        attendee = schemas.AttendeeBase(email=faker.email(), name=faker.name(), timezone='Europe/Berlin')
 
         mailer = ConfirmationMail(confirm_url, deny_url, attendee, now, to=fake_email)
         assert mailer.html()

From 9a7b75294ba7c228a4487d8fb348c55e9c4b605d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20M=C3=BCller?= <mail@devmount.de>
Date: Thu, 18 Apr 2024 17:33:31 +0200
Subject: [PATCH 5/5] =?UTF-8?q?=F0=9F=94=A8=20Fix=20appointment=20tests?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 backend/test/integration/test_appointment.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/backend/test/integration/test_appointment.py b/backend/test/integration/test_appointment.py
index a9f1c12e3..87d3c95a6 100644
--- a/backend/test/integration/test_appointment.py
+++ b/backend/test/integration/test_appointment.py
@@ -338,7 +338,7 @@ def test_attendee_selects_slot_of_unavailable_appointment(self, with_db, with_cl
 
         response = with_client.put(
             f"/apmt/public/{generated_appointment.slug}",
-            json={"slot_id": generated_appointment.slots[-1].id, "attendee": {"email": "a", "name": "b"}},
+            json={"slot_id": generated_appointment.slots[-1].id, "attendee": {"email": "a", "name": "b", "timezone": "c"}},
         )
         assert response.status_code == 403, response.text
 
@@ -347,7 +347,7 @@ def test_attendee_selects_slot_of_missing_appointment(self, with_client, make_ap
 
         response = with_client.put(
             f"/apmt/public/{generated_appointment}",
-            json={"slot_id": generated_appointment.slots[0].id, "attendee": {"email": "a", "name": "b"}},
+            json={"slot_id": generated_appointment.slots[0].id, "attendee": {"email": "a", "name": "b", "timezone": "c"}},
         )
         assert response.status_code == 404, response.text
 
@@ -356,7 +356,7 @@ def test_attendee_selects_missing_slot_of_existing_appointment(self, with_client
 
         response = with_client.put(
             f"/apmt/public/{generated_appointment.id}",
-            json={"slot_id": generated_appointment.slots[0].id + 1, "attendee": {"email": "a", "name": "b"}},
+            json={"slot_id": generated_appointment.slots[0].id + 1, "attendee": {"email": "a", "name": "b", "timezone": "c"}},
         )
         assert response.status_code == 404, response.text