Merge branch 'main' into enhancements/1403-update-booking-confirmation-design

Author: devmount

backend/src/appointment/routes/auth.py

@@ -653,9 +653,15 @@ def oidc_token(
         raise HTTPException(status_code=403, detail=l10n('invalid-credentials'))
 
     oidc_id = token_data.get('sub')
-    username = email = token_data.get('preferred_username', token_data.get('username'))
     name = token_data.get('name')
 
+    # preferred_username is the thundermail address (@thundermail.com)
+    email = token_data.get('preferred_username', token_data.get('username'))
+
+    # the local part of the email should already be unique
+    # so we can strip out the domain portion for a better Booking Link URL
+    username = email.split('@')[0] if email else email
+
     subscriber = repo.external_connection.get_subscriber_by_oidc_id(db, oidc_id)
 
     # If we can't find the subscriber by oidc_id, try to find them by email

backend/test/integration/test_auth.py

@@ -934,3 +934,31 @@ def test_oidc_token_fallback_match_by_email(self, with_db, with_client, make_pro
                 assert oidc_connection is not None
                 assert len(oidc_connection) == 1
                 assert oidc_connection[0].type_id == oidc_id
+
+    def test_oidc_token_strips_domain_from_username(self, with_db, with_client, faker):
+        """Ensure usernames derived from emails drop the domain portion"""
+        os.environ['AUTH_SCHEME'] = 'oidc'
+
+        email = faker.email()
+        expected_username = email.split('@')[0]
+        oidc_id = 'new-oidc-id-strip-domain'
+
+        with patch('appointment.controller.apis.oidc_client.OIDCClient.introspect_token') as mock_introspect:
+            mock_introspect.return_value = {
+                'sub': oidc_id,
+                'email': email,
+                'preferred_username': email, # preferred_username is the thundermail address
+                'name': 'OIDC User',
+            }
+
+            response = with_client.post(
+                '/oidc/token', json={'access_token': 'valid_token', 'timezone': 'America/Vancouver'}
+            )
+
+        assert response.status_code == 200, response.text
+        assert response.json() is True
+
+        with with_db() as db:
+            subscriber = repo.subscriber.get_by_email(db, email)
+            assert subscriber is not None
+            assert subscriber.username == expected_username

backend/test/unit/test_calendar_tools.py

@@ -265,9 +265,8 @@ def test_for_host(self):
         assert ttl
 
     def test_for_txt_record(self):
-        """This domain is used with permission from the owner (me, melissa autumn!)"""
-        host, ttl = Tools.dns_caldav_lookup('melissaautumn.com')
-        assert host == 'https://caldav.fastmail.com:443/dav/'
+        host, ttl = Tools.dns_caldav_lookup('thundermail.com')
+        assert host == 'https://mail.thundermail.com:443/dav/cal/'
         assert ttl
 
     def test_no_records(self):

frontend/src/components/ConfirmationModal.vue

@@ -33,17 +33,16 @@ const emit = defineEmits(['close', 'confirm', 'error']);
     class="position-center fixed z-50 flex w-full max-w-lg flex-col items-center gap-6 rounded-xl bg-white p-12 dark:bg-gray-700"
   >
     <div class="btn-close absolute right-8 top-8 cursor-pointer" @click="emit('close')" :title="t('label.close')">
-      <ph-x class="size-6 fill-transparent stroke-gray-700 stroke-1 dark:stroke-gray-400" />
+      <ph-x class="size-6" />
     </div>
-    <div class="text-2xl font-semibold text-teal-500">
+    <div class="text-2xl font-semibold title-color">
       {{ title }}
     </div>
     <div class="max-w-sm text-center">
       {{ message }}
     </div>
     <div class="flex gap-4">
       <primary-button
-        class="btn-close"
         variant="outline"
         @click="emit('close')"
         :title="cancelLabel"
@@ -69,3 +68,17 @@ const emit = defineEmits(['close', 'confirm', 'error']);
     </div>
   </div>
 </template>
+
+<style scoped>
+  .title-color {
+    color: var(--colour-ti-highlight);
+  }
+
+  .btn-close {
+    color: var(--colour-ti-muted);
+
+    &:hover {
+      color: var(--colour-ti-secondary);
+    }
+  }
+</style>

frontend/src/components/RadioGroupPill.vue

@@ -74,6 +74,7 @@
     "googleCalendarLogo": "Google Kalender Logo",
     "createBookingPage": "Buchungsseite erstellen",
     "bookingPageTitle": "Buchungsseite Titel",
+    "bookingPageInitialValue": "{name}'s Verfügbarkeit",
     "bookingPagePlaceholder": "[Vor- und Nachname]'s Verfügbarkeit",
     "bookingPageTitleHelp": "Dies kann später in der Verfügbarkeit geändert werden.",
     "calendarForNewAppointments": "Kalender für neue Termine",
@@ -101,13 +102,13 @@
     "calendarSettings": "Kalender-Einstellungen",
     "calendarsConnected": "Verbundene Kalender",
     "calendarsUnconnected": "Verfügbare Kalender",
-    "cancelAppointmentService": "Termin-Service abbrechen?",
     "connectedAccounts": "Verbundene Konten",
     "connectedAccountsSettings": "Verbundene Konten Einstellungen",
     "connectedApplications": "Verbundene Anwendungen",
     "contactRequest": "Kontaktanfrage",
     "createNewAppointment": "Neuen Termin erstellen",
     "dateAndTimeFormatting": "Datums- und Zeitformat",
+    "deleteAppointmentData": "Termin-Daten löschen",
     "discoverCaldavcalendars": "CalDAV-Kalender abfragen",
     "displayInformation": "Anzeige-Informationen",
     "eventBooked": "Event gebucht!",
@@ -154,10 +155,8 @@
     "bookingSuccessfullyDenied": "Buchung erfolgreich abgelehnt!",
     "bookingSuccessfullyRequested": "Buchung erfolgreich angefragt!",
     "bookingWillBeCanceled": "Booking will be canceled. An update will be sent to the booker.",
-    "cancelServiceInfo": "Um deinen Appointment Service abzubrechen, klicke hier:\n Hinweis: Dies hat keinen Einfluss auf dein TB Pro Konto oder Abonnement.",
     "connectionEstablished": "Verbindung hergestellt",
     "copiedToClipboard": "In Zwischenablage kopiert",
-    "downloadAccountData": "Um alle Daten herunterzuladen, die vom Appointment Service verwendet werden, klicken Sie hier:",
     "eventWasCreated": "Das Event wurde in deinem Kalender erstellt.",
     "invitationGenerated": "Die Einladungscodes wurden erfolgreich generiert.",
     "invitationWasSent": "Eine Einladung wurde an deine E-Mail-Adresse gesendet.",
@@ -169,8 +168,8 @@
     "slotIsAvailableAgain": "Das Zeitfenster ist jetzt wieder buchbar."
   },
   "label": {
-    "12hAmPm": "12-hour AM/PM",
-    "24h": "24-hour",
+    "12hAmPm": "12:00 Uhr (12-Stunden-Format)",
+    "24h": "24:00 (24-Stunden-Format)",
     "DDMMYYYY": "TT/MM/YYYY",
     "MMDDYYYY": "MM/TT/YYYY",
     "accepted": "Akzeptiert",
@@ -226,8 +225,7 @@
     "bookingDetails": "Buchungsdetails",
     "bookingLink": "Buchungslink",
     "bookingSlot": "Gebuchtes Zeitfenster",
-    "bookingPageURL": "Buchungsseite URL",
-    "bookingPageSettings": "Buchungsseite Einstellungen",
+    "bookingPageLinkLabel": "Buchungsseite Link",
     "bookings": "Buchungen",
     "bookingPageLink": "Dein Buchungsseiten-Link",
     "bookingPageDetails": "Buchungsseite Details",
@@ -242,7 +240,6 @@
     "cancelled": "Storniert",
     "cancelBooking": "Buchung absagen",
     "cancelReason": "Grund für die Absage",
-    "cancelService": "Service abbrechen",
     "changeMyAvailability": "Verfügbarkeit ändern",
     "changesNotSaved": "Änderungen wurden nicht gespeichert",
     "checkAvailableSlots": "Verfügbare Zeiten anzeigen",
@@ -259,7 +256,7 @@
     "connect": "Verbinden",
     "connectCalendar": "Kalender verbinden",
     "connectGoogleCalendar": "Google Kalender verbinden",
-    "connectedAs": "Verbunden als {name}",
+    "connectedAs": "Verbunden als",
     "contact": "Kontakt",
     "continue": "Weiter",
     "contributeToThisSite": "Dieser Seite beitragen",
@@ -287,6 +284,8 @@
     "defaultTimeZone": "Standard Zeitzone",
     "delete": "Löschen",
     "deleteBooking": "Buchung löschen",
+    "deleteAllAppointmentData": "Alle Appointment-Daten löschen",
+    "deleteAllAppointmentDataInfo": "Dein TB Pro-Abonnement bleibt davon unberührt.",
     "deny": "Ablehnen",
     "denyBooking": "Buchung ablehnen",
     "disable": "Deaktivieren",
@@ -298,7 +297,7 @@
     "downloadICS": "Download ICS",
     "downloadInvitation": "Termineinladung herunterladen",
     "downloadTheIcsFile": "Zu Kalender hinzufügen",
-    "downloadMyData": "Meine Daten herunterladen",
+    "downloadMyData": "Alle deine Daten von Appointment herunterladen",
     "earliestBooking": "Früheste Buchung",
     "edit": "Bearbeiten",
     "editCalendar": "Kalender bearbeiten",
@@ -344,6 +343,7 @@
     "logIn": "Anmelden",
     "logOut": "Abmelden",
     "loginToContinue": "Anmelden, um fortzufahren",
+    "manageBookingLink": "Buchungsseite verwalten",
     "meetingDuration": "Meeting-Dauer",
     "meetingTitle": "Meeting Titel",
     "meetingDate": "Meeting-Datum",
@@ -435,7 +435,7 @@
     "sortBy": "Sortieren nach",
     "start": "Start",
     "startDate": "Startdatum",
-    "startOfWeek": "Wochenbeginn",
+    "startOfWeek": "Wochentag als Wochenbeginn",
     "startTime": "Startzeit",
     "startUsingTba": "Starte mit TBA",
     "status": "Status",
@@ -540,7 +540,6 @@
     "zoomCom": "zoom.com …"
   },
   "text": {
-    "accountCancelWarning": "Du wirst keinen Zugriff mehr auf Thunderbird Appointment haben. Alle Buchungs-Kalender werden auf \"Inaktiv\" gesetzt und alle zukünftigen Buchungen werden storniert.\nAndere Termin-Daten werden 30 Tage lang beibehalten, entsprechend der TB Pro Datenschutzrichtlinie.",
     "admin": {
       "completelyRemoveUser": "Hier wird ein Benutzer vollständig aus Thunderbird Appointment entfernt, einschließlich aller Benutzerdaten. Diese Funktion bitte nicht ausführen, wenn es keine Aufforderung dafür gab.",
       "disableAccountFirst": "Das Konto muss zunächst deaktiviert sein."
@@ -550,6 +549,7 @@
     "chooseDateAndTime": "Wähle Tag und Zeit für ein Treffen.",
     "connectZoom": "Du kannst dein Zoom-Konto verbinden, um Besprechungen direkt mit einer Zoom-Einladung zu erstellen.",
     "contactRequestForm": "Bitte verwende das nachstehende Kontaktformular, um Feedback, Fragen oder Anliegen an unser Support-Team zu senden. Wir werden ggf. versuchen, so schnell wie möglich zu antworten.",
+    "deleteAppointmentDataWarning": "Du wirst keinen Zugriff mehr auf Thunderbird Appointment haben. Alle Buchungs-Kalender werden auf \"Inaktiv\" gesetzt und alle zukünftigen Buchungen werden storniert.\nAndere Termin-Daten werden 30 Tage lang beibehalten, entsprechend der TB Pro Datenschutzrichtlinie.",
     "defineDaysAndTimeSlots": "Definiere Tage und Zeitfenster, damit Teilnehmer die Zeit des Events auswählen können …",
     "denialSentToAddress": "Die Informationen über die Ablehnung der Anfrage wurde per E-Mail an {address} geschickt.",
     "disclaimerGABooking": "Bitte beachten: Der Eigentümer muss die Buchung bestätigen.",