From 47896444a6a5f71bf726e481675fe9f66eda88bd Mon Sep 17 00:00:00 2001
From: Daniel Egger <daniel.egger@gmail.com>
Date: Thu, 4 Jul 2024 17:38:28 +0200
Subject: [PATCH 01/12] Make `ItDatePicker` component

---
 .../components/onboarding/PersonalAddress.vue | 38 +-----------
 .../personalProfile/ProfileEdit.vue           |  4 ++
 client/src/components/ui/ItDatePicker.vue     | 59 +++++++++++++++++++
 3 files changed, 65 insertions(+), 36 deletions(-)
 create mode 100644 client/src/components/ui/ItDatePicker.vue

diff --git a/client/src/components/onboarding/PersonalAddress.vue b/client/src/components/onboarding/PersonalAddress.vue
index e40dd405..c4fcbeea 100644
--- a/client/src/components/onboarding/PersonalAddress.vue
+++ b/client/src/components/onboarding/PersonalAddress.vue
@@ -1,10 +1,9 @@
 <script setup lang="ts">
 import { computed } from "vue";
 import { useEntities } from "@/services/entities";
-import VueDatePicker from "@vuepic/vue-datepicker";
 import "@vuepic/vue-datepicker/dist/main.css";
 import { t } from "i18next";
-import { useUserStore } from "@/stores/user";
+import ItDatePicker from "@/components/ui/ItDatePicker.vue";
 
 const props = defineProps<{
   modelValue: {
@@ -25,7 +24,6 @@ const props = defineProps<{
 const emit = defineEmits(["update:modelValue"]);
 
 const { countries } = useEntities();
-const userStore = useUserStore();
 
 const paymentMethods = [
   { value: "credit_card", label: t("a.Debit-/Kreditkarte/Twint") },
@@ -234,40 +232,8 @@ const address = computed({
         {{ $t("a.Geburtsdatum") }}
       </label>
       <div class="mt-2">
-        <VueDatePicker
-          v-model="address.birth_date"
-          format="dd.MM.yyyy"
-          no-today
-          model-type="yyyy-MM-dd"
-          name="birth-date"
-          max-date="2007-01-01"
-          prevent-min-max-navigation
-          required
-          :enable-time-picker="false"
-          text-input
-          placeholder="15.06.1982"
-          start-date="1982-01-01"
-          :locale="userStore.language"
-          :cancel-text="$t('a.Abbrechen')"
-          :select-text="$t('a.Auswählen')"
-        ></VueDatePicker>
+        <ItDatePicker v-model="address.birth_date"></ItDatePicker>
       </div>
     </div>
   </div>
 </template>
-
-<style>
-/* Theming for date picker */
-.dp__theme_light {
-  --dp-text-color: #585f63;
-  --dp-primary-color: #41b5fa;
-  --dp-border-color-focus: #3d6dcc;
-}
-
-:root {
-  --dp-font-family: "Buenos Aires" sans-serif;
-  --dp-border-radius: none;
-  --dp-font-size: 0.875rem;
-  --dp-cell-border-radius: none;
-}
-</style>
diff --git a/client/src/components/personalProfile/ProfileEdit.vue b/client/src/components/personalProfile/ProfileEdit.vue
index 77d0da8d..6a928e1b 100644
--- a/client/src/components/personalProfile/ProfileEdit.vue
+++ b/client/src/components/personalProfile/ProfileEdit.vue
@@ -21,6 +21,10 @@ const formData = ref({
   postal_code: user.postal_code,
   city: user.city,
   country_code: user.country?.country_code,
+
+  phone_number: user.phone_number,
+  birth_date: user.birth_date,
+
   organisation: user.organisation,
   organisation_street: user.organisation_street,
   organisation_street_number: user.organisation_street_number,
diff --git a/client/src/components/ui/ItDatePicker.vue b/client/src/components/ui/ItDatePicker.vue
new file mode 100644
index 00000000..4ded7f00
--- /dev/null
+++ b/client/src/components/ui/ItDatePicker.vue
@@ -0,0 +1,59 @@
+<script setup lang="ts">
+import { useUserStore } from "@/stores/user";
+import VueDatePicker from "@vuepic/vue-datepicker";
+import { defineProps, withDefaults, defineModel } from "vue";
+
+const model = defineModel<string>();
+
+export interface Props {
+  noToday?: boolean;
+  required?: boolean;
+  placeholder?: string;
+  startDate?: string;
+  maxDate?: string;
+}
+
+const props = withDefaults(defineProps<Props>(), {
+  required: false,
+  placeholder: "15.06.1982",
+  startDate: "1982-01-01",
+  maxDate: "2007-01-01",
+});
+
+const userStore = useUserStore();
+</script>
+
+<template>
+  <VueDatePicker
+    v-model="model"
+    format="dd.MM.yyyy"
+    model-type="yyyy-MM-dd"
+    name="date"
+    prevent-min-max-navigation
+    :required="props.required"
+    :enable-time-picker="false"
+    text-input
+    :no-today="props.noToday"
+    :max-date="props.maxDate"
+    :placeholder="props.placeholder"
+    :start-date="props.startDate"
+    :locale="userStore.language"
+    :cancel-text="$t('a.Abbrechen')"
+    :select-text="$t('a.Auswählen')"
+  ></VueDatePicker>
+</template>
+
+<style>
+.dp__theme_light {
+  --dp-text-color: #585f63;
+  --dp-primary-color: #41b5fa;
+  --dp-border-color-focus: #3d6dcc;
+}
+
+:root {
+  --dp-font-family: "Buenos Aires" sans-serif;
+  --dp-border-radius: none;
+  --dp-font-size: 0.875rem;
+  --dp-cell-border-radius: none;
+}
+</style>

From d56c346512fb33f889496f6c5df9b678a6de8381 Mon Sep 17 00:00:00 2001
From: Daniel Egger <daniel.egger@gmail.com>
Date: Tue, 16 Jul 2024 10:58:58 +0200
Subject: [PATCH 02/12] Add more editable profile fields

---
 .../personalProfile/ProfileEdit.vue           | 48 +++++++++-
 .../personalProfile/ProfileView.vue           | 95 ++++++++++++-------
 client/src/components/ui/ItDatePicker.vue     |  1 +
 client/src/pages/StyleGuidePage.vue           |  9 ++
 server/vbv_lernwelt/core/models.py            |  3 +-
 5 files changed, 121 insertions(+), 35 deletions(-)

diff --git a/client/src/components/personalProfile/ProfileEdit.vue b/client/src/components/personalProfile/ProfileEdit.vue
index 6a928e1b..e668857a 100644
--- a/client/src/components/personalProfile/ProfileEdit.vue
+++ b/client/src/components/personalProfile/ProfileEdit.vue
@@ -3,6 +3,8 @@ import { useEntities } from "@/services/entities";
 import AvatarImage from "@/components/ui/AvatarImage.vue";
 import { ref } from "vue";
 import { type User, useUserStore } from "@/stores/user";
+import ItDatePicker from "@/components/ui/ItDatePicker.vue";
+import { normalizeSwissPhoneNumber } from "@/utils/phone";
 
 const emit = defineEmits(["cancel", "save"]);
 
@@ -26,6 +28,7 @@ const formData = ref({
   birth_date: user.birth_date,
 
   organisation: user.organisation,
+  organisation_detail_name: user.organisation_detail_name,
   organisation_street: user.organisation_street,
   organisation_street_number: user.organisation_street_number,
   organisation_postal_code: user.organisation_postal_code,
@@ -35,7 +38,8 @@ const formData = ref({
 });
 
 async function save() {
-  const { country_code, organisation_country_code, ...profileData } = formData.value;
+  const { country_code, organisation_country_code, phone_number, ...profileData } =
+    formData.value;
   const typedProfileData: Partial<User> = { ...profileData };
 
   typedProfileData.country = countries.value.find(
@@ -45,6 +49,10 @@ async function save() {
     (c) => c.country_code === organisation_country_code
   );
 
+  if (phone_number) {
+    typedProfileData.phone_number = normalizeSwissPhoneNumber(phone_number);
+  }
+
   await user.updateUserProfile(typedProfileData);
   emit("save");
 }
@@ -130,6 +138,28 @@ async function avatarUpload(e: Event) {
         disabled
         class="disabled:bg-gray-50 mb-4 block w-full border-0 py-1.5 text-gray-900 ring-1 ring-inset ring-gray-300 placeholder:text-gray-400 focus:ring-2 focus:ring-inset focus:ring-blue-600 disabled:cursor-not-allowed disabled:text-gray-500 disabled:ring-gray-200 sm:max-w-sm sm:text-sm sm:leading-6"
       />
+
+      <label for="phone" class="block pb-1.5 leading-6">
+        {{ $t("a.Telefonnummer") }}
+      </label>
+      <div>
+        <input
+          id="phone"
+          v-model="formData.phone_number"
+          type="text"
+          name="phone"
+          autocomplete="phone-number"
+          class="disabled:bg-gray-50 mb-4 block w-full border-0 py-1.5 text-gray-900 ring-1 ring-inset ring-gray-300 placeholder:text-gray-400 focus:ring-2 focus:ring-inset focus:ring-blue-600 disabled:cursor-not-allowed disabled:text-gray-500 disabled:ring-gray-200 sm:max-w-sm sm:text-sm sm:leading-6"
+        />
+      </div>
+
+      <label for="birth-date" class="block pb-1.5 leading-6">
+        {{ $t("a.Geburtsdatum") }}
+      </label>
+      <div class="mb-4 block w-full py-1.5 sm:max-w-sm sm:text-sm sm:leading-6">
+        <ItDatePicker v-model="formData.birth_date"></ItDatePicker>
+      </div>
+
       <label class="block pb-1.5 leading-6">
         {{ $t("a.Profilbild") }}
       </label>
@@ -268,6 +298,22 @@ async function avatarUpload(e: Event) {
         {{ $t("a.Firmenanschrift") }}
       </h4>
 
+      <div class="flex flex-col justify-start md:flex-row md:space-x-4">
+        <div class="w-full md:max-w-lg">
+          <label for="org-street-address" class="block pb-1.5 leading-6">
+            {{ $t("a.Name") }}
+          </label>
+
+          <input
+            id="org-detail-name"
+            v-model="formData.organisation_detail_name"
+            type="text"
+            name="org-detail-name"
+            class="disabled:bg-gray-50 mb-4 block w-full border-0 py-1.5 text-gray-900 ring-1 ring-inset ring-gray-300 placeholder:text-gray-400 focus:ring-2 focus:ring-inset focus:ring-blue-600 disabled:cursor-not-allowed disabled:text-gray-500 disabled:ring-gray-200 sm:text-sm sm:leading-6"
+          />
+        </div>
+      </div>
+
       <div class="flex flex-col justify-start md:flex-row md:space-x-4">
         <div class="w-full md:max-w-sm">
           <label for="org-street-address" class="block pb-1.5 leading-6">
diff --git a/client/src/components/personalProfile/ProfileView.vue b/client/src/components/personalProfile/ProfileView.vue
index b5d2d06e..b41779c7 100644
--- a/client/src/components/personalProfile/ProfileView.vue
+++ b/client/src/components/personalProfile/ProfileView.vue
@@ -3,6 +3,8 @@ import { useUserStore } from "@/stores/user";
 import { computed } from "vue";
 import { useEntities } from "@/services/entities";
 import { useTranslation } from "i18next-vue";
+import dayjs from "dayjs";
+import { displaySwissPhoneNumber } from "@/utils/phone";
 
 const { t } = useTranslation();
 
@@ -10,21 +12,20 @@ const user = useUserStore();
 const { organisations } = useEntities();
 
 const privateAddress = computed(() => {
-  let addressText = `${user.street} ${user.street_number}`.trim();
-  if (user.postal_code || user.city) {
-    if (addressText.length) {
-      addressText += ", ";
-    }
-    addressText += `${user.postal_code} ${user.city}`;
-  }
-  if (user.country) {
-    if (addressText.length) {
-      addressText += ", ";
-    }
-    addressText += user.country.name;
+  const textParts = [];
+
+  if (user.street || user.street_number) {
+    textParts.push(`${user.street} ${user.street_number}`.trim());
   }
 
-  return addressText.trim();
+  if (user.postal_code || user.city) {
+    textParts.push(`${user.postal_code} ${user.city}`);
+  }
+  if (textParts.length && user.country) {
+    textParts.push(user.country.name);
+  }
+
+  return textParts;
 });
 
 const organisationName = computed(() => {
@@ -36,22 +37,25 @@ const organisationName = computed(() => {
 });
 
 const orgAddress = computed(() => {
-  let addressText =
-    `${user.organisation_street} ${user.organisation_street_number}`.trim();
-  if (user.organisation_postal_code || user.organisation_city) {
-    if (addressText.length) {
-      addressText += ", ";
-    }
-    addressText += `${user.organisation_postal_code} ${user.organisation_city}`;
-  }
-  if (user.organisation_country) {
-    if (addressText.length) {
-      addressText += ", ";
-    }
-    addressText += user.organisation_country.name;
+  const textParts = [];
+
+  if (user.organisation_detail_name) {
+    textParts.push(user.organisation_detail_name);
   }
 
-  return addressText.trim();
+  if (user.organisation_street || user.organisation_street_number) {
+    textParts.push(
+      `${user.organisation_street} ${user.organisation_street_number}`.trim()
+    );
+  }
+  if (user.organisation_postal_code || user.organisation_city) {
+    textParts.push(`${user.organisation_postal_code} ${user.organisation_city}`);
+  }
+  if (textParts.length && user.organisation_country) {
+    textParts.push(user.organisation_country.name);
+  }
+
+  return textParts;
 });
 
 const invoiceAddress = computed(() => {
@@ -74,13 +78,34 @@ const invoiceAddress = computed(() => {
         {{ $t("a.E-Mail Adresse") }}
       </label>
       <div class="mb-3 sm:col-span-2 sm:mb-0">{{ user.email }}</div>
+      <label class="block font-semibold leading-6">
+        {{ $t("a.Telefonnummer") }}
+      </label>
+      <div class="mb-3 sm:col-span-2 sm:mb-0">
+        <span v-if="user.phone_number">
+          {{ displaySwissPhoneNumber(user.phone_number) }}
+        </span>
+        <span v-else class="text-gray-800">{{ $t("a.Keine Angabe") }}</span>
+      </div>
+      <label class="block font-semibold leading-6">
+        {{ $t("a.Geburtsdatum") }}
+      </label>
+      <div class="mb-3 sm:col-span-2 sm:mb-0">
+        <span v-if="user.birth_date">
+          {{ dayjs(user.birth_date).format("DD.MM.YYYY") }}
+        </span>
+        <span v-else class="text-gray-800">{{ $t("a.Keine Angabe") }}</span>
+      </div>
       <label class="block font-semibold leading-6">
         {{ $t("a.Privatadresse") }}
       </label>
       <div class="mb-3 sm:col-span-2 sm:mb-0">
-        <template v-if="privateAddress">
-          {{ privateAddress }}
-        </template>
+        <div v-if="privateAddress.length">
+          <span v-for="(line, index) in privateAddress" :key="index">
+            {{ line }}
+            <br />
+          </span>
+        </div>
         <span v-else class="text-gray-800">{{ $t("a.Keine Angabe") }}</span>
       </div>
     </div>
@@ -94,9 +119,13 @@ const invoiceAddress = computed(() => {
         {{ $t("a.Firmenanschrift") }}
       </label>
       <div class="sm:col-span-2">
-        <template v-if="orgAddress">
-          {{ orgAddress }}
-        </template>
+        <!-- eslint-disable vue/no-v-html -->
+        <div v-if="orgAddress">
+          <span v-for="(line, index) in orgAddress" :key="index">
+            {{ line }}
+            <br />
+          </span>
+        </div>
         <span v-else class="text-gray-800">{{ $t("a.Keine Angabe") }}</span>
       </div>
     </div>
diff --git a/client/src/components/ui/ItDatePicker.vue b/client/src/components/ui/ItDatePicker.vue
index 4ded7f00..4c44f46d 100644
--- a/client/src/components/ui/ItDatePicker.vue
+++ b/client/src/components/ui/ItDatePicker.vue
@@ -1,6 +1,7 @@
 <script setup lang="ts">
 import { useUserStore } from "@/stores/user";
 import VueDatePicker from "@vuepic/vue-datepicker";
+import "@vuepic/vue-datepicker/dist/main.css";
 import { defineProps, withDefaults, defineModel } from "vue";
 
 const model = defineModel<string>();
diff --git a/client/src/pages/StyleGuidePage.vue b/client/src/pages/StyleGuidePage.vue
index 1cb4a1e3..26f791bb 100644
--- a/client/src/pages/StyleGuidePage.vue
+++ b/client/src/pages/StyleGuidePage.vue
@@ -13,6 +13,8 @@ import VerticalBarChart from "@/components/ui/VerticalBarChart.vue";
 import LearningPathCircle from "@/pages/learningPath/learningPathPage/LearningPathCircle.vue";
 import logger from "loglevel";
 import { reactive, ref } from "vue";
+import VueDatePicker from "@vuepic/vue-datepicker";
+import "@vuepic/vue-datepicker/dist/main.css";
 
 const state = reactive({
   checkboxValue: true,
@@ -34,6 +36,8 @@ const state = reactive({
   },
 });
 
+const birtDate = ref("1982-06-15");
+
 const dropdownData = [
   {
     title: "Option 1",
@@ -411,6 +415,11 @@ function log(data: any) {
     ></ItDropdownSelect>
     {{ state.dropdownSelected }}
 
+    <h2 class="mb-8 mt-8">Date Picker</h2>
+    <div class="mt-2">
+      <VueDatePicker v-model="birthDate"></VueDatePicker>
+    </div>
+
     <h2 class="mb-8 mt-8">Checkbox</h2>
 
     <ItCheckbox
diff --git a/server/vbv_lernwelt/core/models.py b/server/vbv_lernwelt/core/models.py
index a1b09008..9d0ca936 100644
--- a/server/vbv_lernwelt/core/models.py
+++ b/server/vbv_lernwelt/core/models.py
@@ -114,8 +114,9 @@ class User(AbstractUser):
         blank=True,
     )
 
-    # fields gathered from cembra pay form
     birth_date = models.DateField(null=True, blank=True)
+
+    # phone number should be stored in the format +41792018586 (not validated)
     phone_number = models.CharField(max_length=255, blank=True, default="")
 
     # is only set by abacus invoice export code

From f0acdaf2549ecd30bdbe3d3ed212cbcc03e668cc Mon Sep 17 00:00:00 2001
From: Daniel Egger <daniel.egger@gmail.com>
Date: Tue, 16 Jul 2024 14:51:44 +0200
Subject: [PATCH 03/12] Add custom organisation name

---
 .../onboarding/OrganisationAddress.vue        |  2 +-
 client/src/consts.ts                          | 19 ++++++
 .../src/pages/onboarding/AccountProfile.vue   | 60 +++++++++++++------
 .../pages/onboarding/vv/CheckoutAddress.vue   | 19 ++++--
 client/src/services/onboarding.ts             | 13 ++++
 server/vbv_lernwelt/course/consts.py          |  7 +++
 6 files changed, 96 insertions(+), 24 deletions(-)
 create mode 100644 client/src/consts.ts

diff --git a/client/src/components/onboarding/OrganisationAddress.vue b/client/src/components/onboarding/OrganisationAddress.vue
index b8b59889..e56e33cd 100644
--- a/client/src/components/onboarding/OrganisationAddress.vue
+++ b/client/src/components/onboarding/OrganisationAddress.vue
@@ -34,7 +34,7 @@ const orgAddress = computed({
         for="company-name"
         class="block text-sm font-medium leading-6 text-gray-900"
       >
-        {{ $t("a.Name") }}
+        {{ $t("a.Firmenname") }}
       </label>
       <div class="mt-2">
         <input
diff --git a/client/src/consts.ts b/client/src/consts.ts
new file mode 100644
index 00000000..25801247
--- /dev/null
+++ b/client/src/consts.ts
@@ -0,0 +1,19 @@
+// Course IDs
+export const COURSE_TEST_ID = -1;
+export const COURSE_UK = -3;
+export const COURSE_VERSICHERUNGSVERMITTLERIN_ID = -4;
+export const COURSE_UK_FR = -5;
+export const COURSE_UK_TRAINING = -6;
+export const COURSE_UK_TRAINING_FR = -7;
+export const COURSE_UK_IT = -8;
+export const COURSE_UK_TRAINING_IT = -9;
+export const COURSE_VERSICHERUNGSVERMITTLERIN_FR_ID = -10;
+export const COURSE_VERSICHERUNGSVERMITTLERIN_IT_ID = -11;
+export const COURSE_VERSICHERUNGSVERMITTLERIN_PRUEFUNG_ID = -12;
+export const COURSE_MOTORFAHRZEUG_PRUEFUNG_ID = -13;
+
+// Organization IDs
+export const ORGANISATION_OTHER_BROKER_ID = 1;
+export const ORGANISATION_OTHER_HEALTH_INSURANCE_ID = 2;
+export const ORGANISATION_OTHER_PRIVATE_INSURANCE_ID = 3;
+export const ORGANISATION_NO_COMPANY_ID = 31;
\ No newline at end of file
diff --git a/client/src/pages/onboarding/AccountProfile.vue b/client/src/pages/onboarding/AccountProfile.vue
index 4a0d780b..2982a9a3 100644
--- a/client/src/pages/onboarding/AccountProfile.vue
+++ b/client/src/pages/onboarding/AccountProfile.vue
@@ -3,9 +3,9 @@ import WizardPage from "@/components/onboarding/WizardPage.vue";
 import ItDropdownSelect from "@/components/ui/ItDropdownSelect.vue";
 import { computed, ref, watch } from "vue";
 import { useUserStore } from "@/stores/user";
-import { useRoute } from "vue-router";
+import { useRoute, useRouter } from "vue-router";
 import { useTranslation } from "i18next-vue";
-import { profileNextRoute } from "@/services/onboarding";
+import { isOtherOrganisation, profileNextRoute } from "@/services/onboarding";
 import { useEntities } from "@/services/entities";
 import AvatarImage from "@/components/ui/AvatarImage.vue";
 
@@ -13,9 +13,12 @@ const { t } = useTranslation();
 
 const user = useUserStore();
 const route = useRoute();
+const router = useRouter();
 
 const { organisations } = useEntities();
 
+const organisationDetailName = ref<string>("");
+
 const selectedOrganisation = ref({
   id: 0,
   name: t("a.Auswählen"),
@@ -35,7 +38,11 @@ watch(
 );
 
 const validOrganisation = computed(() => {
-  return selectedOrganisation.value.id !== 0;
+  const organisationSelected = selectedOrganisation.value.id !== 0;
+  const organisationNameSet =
+    !isOtherOrganisation(selectedOrganisation.value.id) ||
+    !!organisationDetailName.value.trim();
+  return organisationSelected && organisationNameSet;
 });
 
 const avatarError = ref(false);
@@ -56,15 +63,21 @@ async function avatarUpload(e: Event) {
   }
 }
 
-watch(selectedOrganisation, async (organisation) => {
+async function updateUserProfile() {
   await user.updateUserProfile({
-    organisation: organisation.id,
+    organisation: selectedOrganisation.value.id,
+    organisation_detail_name: organisationDetailName.value.trim(),
   });
-});
+}
 
 const nextRoute = computed(() => {
   return profileNextRoute(route.params.courseType);
 });
+
+async function navigateNextRoute() {
+  await updateUserProfile();
+  await router.push({ name: nextRoute.value });
+}
 </script>
 
 <template>
@@ -86,6 +99,19 @@ const nextRoute = computed(() => {
 
       <ItDropdownSelect v-model="selectedOrganisation" :items="organisations" />
 
+      <div v-if="isOtherOrganisation(selectedOrganisation.id)" class="my-8">
+        <label for="organisationDetailName" class="block pb-1.5 heading-3">
+          {{ $t("a.Firmenname") }}
+        </label>
+        <input
+          id="organisationDetailName"
+          v-model="organisationDetailName"
+          type="text"
+          name="phone"
+          class="block w-full border-0 py-1.5 text-gray-900 ring-1 ring-inset ring-gray-300 placeholder:text-gray-400 focus:ring-2 focus:ring-inset focus:ring-blue-600 disabled:cursor-not-allowed disabled:text-gray-500 disabled:ring-gray-200 sm:max-w-sm sm:text-sm sm:leading-6"
+        />
+      </div>
+
       <div class="mt-16 flex flex-col justify-between gap-12 lg:flex-row lg:gap-24">
         <div>
           <h3 class="mb-3">{{ $t("a.Profilbild") }}</h3>
@@ -117,18 +143,16 @@ const nextRoute = computed(() => {
     </template>
 
     <template #footer>
-      <router-link v-slot="{ navigate }" :to="{ name: nextRoute }" custom>
-        <button
-          :disabled="!validOrganisation"
-          class="btn-blue flex items-center"
-          role="link"
-          data-cy="continue-button"
-          @click="navigate"
-        >
-          {{ $t("general.next") }}
-          <it-icon-arrow-right class="it-icon ml-2 h-6 w-6" />
-        </button>
-      </router-link>
+      <button
+        :disabled="!validOrganisation"
+        class="btn-blue flex items-center"
+        role="link"
+        data-cy="continue-button"
+        @click="navigateNextRoute"
+      >
+        {{ $t("general.next") }}
+        <it-icon-arrow-right class="it-icon ml-2 h-6 w-6" />
+      </button>
     </template>
   </WizardPage>
 </template>
diff --git a/client/src/pages/onboarding/vv/CheckoutAddress.vue b/client/src/pages/onboarding/vv/CheckoutAddress.vue
index f8ff17ef..76e5a6ed 100644
--- a/client/src/pages/onboarding/vv/CheckoutAddress.vue
+++ b/client/src/pages/onboarding/vv/CheckoutAddress.vue
@@ -13,6 +13,12 @@ import DatatransCembraDeviceFingerprint from "@/components/onboarding/DatatransC
 import { getLocalSessionKey } from "@/statistics";
 import log from "loglevel";
 import { normalizeSwissPhoneNumber, validatePhoneNumber } from "@/utils/phone";
+import {
+  ORGANISATION_NO_COMPANY_ID,
+  ORGANISATION_OTHER_BROKER_ID,
+  ORGANISATION_OTHER_HEALTH_INSURANCE_ID,
+  ORGANISATION_OTHER_PRIVATE_INSURANCE_ID,
+} from "@/consts";
 
 const props = defineProps({
   courseType: {
@@ -31,11 +37,14 @@ const userOrganisationName = computed(() => {
   }
 
   // Those IDs do not represent a company
-  // 1: Other broker
-  // 2: Other insurance
-  // 3: Other private insurance
-  // 31: No company relation
-  if ([1, 2, 3, 31].includes(user.organisation)) {
+  if (
+    [
+      ORGANISATION_OTHER_BROKER_ID,
+      ORGANISATION_OTHER_HEALTH_INSURANCE_ID,
+      ORGANISATION_OTHER_PRIVATE_INSURANCE_ID,
+      ORGANISATION_NO_COMPANY_ID,
+    ].includes(user.organisation)
+  ) {
     return null;
   }
 
diff --git a/client/src/services/onboarding.ts b/client/src/services/onboarding.ts
index 24b59a48..9c59ef48 100644
--- a/client/src/services/onboarding.ts
+++ b/client/src/services/onboarding.ts
@@ -1,4 +1,9 @@
 import { isString, startsWith } from "lodash";
+import {
+  ORGANISATION_OTHER_BROKER_ID,
+  ORGANISATION_OTHER_HEALTH_INSURANCE_ID,
+  ORGANISATION_OTHER_PRIVATE_INSURANCE_ID,
+} from "@/consts";
 
 export function profileNextRoute(courseType: string | string[]) {
   if (courseType === "uk") {
@@ -10,3 +15,11 @@ export function profileNextRoute(courseType: string | string[]) {
   }
   return "";
 }
+
+export function isOtherOrganisation(orgId: number) {
+  return [
+    ORGANISATION_OTHER_BROKER_ID,
+    ORGANISATION_OTHER_HEALTH_INSURANCE_ID,
+    ORGANISATION_OTHER_PRIVATE_INSURANCE_ID,
+  ].includes(orgId);
+}
diff --git a/server/vbv_lernwelt/course/consts.py b/server/vbv_lernwelt/course/consts.py
index a0dd7b27..a9949458 100644
--- a/server/vbv_lernwelt/course/consts.py
+++ b/server/vbv_lernwelt/course/consts.py
@@ -28,3 +28,10 @@ UK_COURSE_IDS = [
     COURSE_UK_TRAINING_FR,
     COURSE_UK_TRAINING_IT,
 ]
+
+
+# Organization IDs
+ORGANISATION_OTHER_BROKER_ID = 1
+ORGANISATION_OTHER_HEALTH_INSURANCE_ID = 2
+ORGANISATION_OTHER_PRIVATE_INSURANCE_ID = 3
+ORGANISATION_NO_COMPANY_ID = 31

From aa30dadfd788f78acb1d866030eb3b922ddf59ef Mon Sep 17 00:00:00 2001
From: Daniel Egger <daniel.egger@gmail.com>
Date: Tue, 16 Jul 2024 15:20:25 +0200
Subject: [PATCH 04/12] Update cypress tests

---
 .../personalProfile/ProfileEdit.vue           |   4 +-
 .../personalProfile/ProfileView.vue           |  23 ++--
 client/src/consts.ts                          |   2 +-
 client/src/pages/StyleGuidePage.vue           |   8 --
 .../src/pages/onboarding/AccountProfile.vue   |   2 +-
 .../personalProfile/PersonalProfilePage.vue   |   6 +-
 cypress/e2e/checkout-vv/checkout.cy.js        | 121 +++++++++++++-----
 cypress/e2e/profile/profile.cy.js             |  85 ++++++++++++
 cypress/support/commands.js                   |  11 ++
 server/vbv_lernwelt/core/serializers.py       |   6 +
 10 files changed, 212 insertions(+), 56 deletions(-)
 create mode 100644 cypress/e2e/profile/profile.cy.js

diff --git a/client/src/components/personalProfile/ProfileEdit.vue b/client/src/components/personalProfile/ProfileEdit.vue
index e668857a..398e363a 100644
--- a/client/src/components/personalProfile/ProfileEdit.vue
+++ b/client/src/components/personalProfile/ProfileEdit.vue
@@ -301,7 +301,7 @@ async function avatarUpload(e: Event) {
       <div class="flex flex-col justify-start md:flex-row md:space-x-4">
         <div class="w-full md:max-w-lg">
           <label for="org-street-address" class="block pb-1.5 leading-6">
-            {{ $t("a.Name") }}
+            {{ $t("a.Firmenname") }}
           </label>
 
           <input
@@ -439,7 +439,7 @@ async function avatarUpload(e: Event) {
     <button class="btn btn-secondary" @click="emit('cancel')">
       {{ $t("general.cancel") }}
     </button>
-    <button class="btn btn-primary" @click="save">
+    <button class="btn btn-primary" data-cy="saveButton" @click="save">
       {{ $t("general.save") }}
     </button>
   </div>
diff --git a/client/src/components/personalProfile/ProfileView.vue b/client/src/components/personalProfile/ProfileView.vue
index b41779c7..ac2e494c 100644
--- a/client/src/components/personalProfile/ProfileView.vue
+++ b/client/src/components/personalProfile/ProfileView.vue
@@ -71,17 +71,21 @@ const invoiceAddress = computed(() => {
     <h3 class="mb-2">{{ $t("a.Persönliche Informationen") }}</h3>
     <div class="sm:grid sm:grid-cols-3 sm:items-start sm:gap-8 sm:py-6">
       <label class="block font-semibold leading-6">{{ $t("a.Vorname") }}</label>
-      <div class="mb-3 sm:col-span-2 sm:mb-0">{{ user.first_name }}</div>
+      <div class="mb-3 sm:col-span-2 sm:mb-0" data-cy="firstName">
+        {{ user.first_name }}
+      </div>
       <label class="block font-semibold leading-6">{{ $t("a.Name") }}</label>
-      <div class="mb-3 sm:col-span-2 sm:mb-0">{{ user.last_name }}</div>
+      <div class="mb-3 sm:col-span-2 sm:mb-0" data-cy="lastName">
+        {{ user.last_name }}
+      </div>
       <label class="block font-semibold leading-6">
         {{ $t("a.E-Mail Adresse") }}
       </label>
-      <div class="mb-3 sm:col-span-2 sm:mb-0">{{ user.email }}</div>
+      <div class="mb-3 sm:col-span-2 sm:mb-0" data-cy="email">{{ user.email }}</div>
       <label class="block font-semibold leading-6">
         {{ $t("a.Telefonnummer") }}
       </label>
-      <div class="mb-3 sm:col-span-2 sm:mb-0">
+      <div class="mb-3 sm:col-span-2 sm:mb-0" data-cy="phone">
         <span v-if="user.phone_number">
           {{ displaySwissPhoneNumber(user.phone_number) }}
         </span>
@@ -90,7 +94,7 @@ const invoiceAddress = computed(() => {
       <label class="block font-semibold leading-6">
         {{ $t("a.Geburtsdatum") }}
       </label>
-      <div class="mb-3 sm:col-span-2 sm:mb-0">
+      <div class="mb-3 sm:col-span-2 sm:mb-0" data-cy="birthDate">
         <span v-if="user.birth_date">
           {{ dayjs(user.birth_date).format("DD.MM.YYYY") }}
         </span>
@@ -99,7 +103,7 @@ const invoiceAddress = computed(() => {
       <label class="block font-semibold leading-6">
         {{ $t("a.Privatadresse") }}
       </label>
-      <div class="mb-3 sm:col-span-2 sm:mb-0">
+      <div class="mb-3 sm:col-span-2 sm:mb-0" data-cy="privateAddress">
         <div v-if="privateAddress.length">
           <span v-for="(line, index) in privateAddress" :key="index">
             {{ line }}
@@ -114,13 +118,14 @@ const invoiceAddress = computed(() => {
     <h3 class="my-2">{{ $t("a.Geschäftsdaten") }}</h3>
     <div class="sm:grid sm:grid-cols-3 sm:items-start sm:gap-8 sm:py-6">
       <label class="block font-semibold leading-6">{{ $t("a.Unternehmen") }}</label>
-      <div class="mb-3 sm:col-span-2 sm:mb-0">{{ organisationName }}</div>
+      <div class="mb-3 sm:col-span-2 sm:mb-0" data-cy="organisationDetailName">
+        {{ organisationName }}
+      </div>
       <label class="block font-semibold leading-6">
         {{ $t("a.Firmenanschrift") }}
       </label>
       <div class="sm:col-span-2">
-        <!-- eslint-disable vue/no-v-html -->
-        <div v-if="orgAddress">
+        <div v-if="orgAddress" data-cy="organisationAddress">
           <span v-for="(line, index) in orgAddress" :key="index">
             {{ line }}
             <br />
diff --git a/client/src/consts.ts b/client/src/consts.ts
index 25801247..656714c7 100644
--- a/client/src/consts.ts
+++ b/client/src/consts.ts
@@ -16,4 +16,4 @@ export const COURSE_MOTORFAHRZEUG_PRUEFUNG_ID = -13;
 export const ORGANISATION_OTHER_BROKER_ID = 1;
 export const ORGANISATION_OTHER_HEALTH_INSURANCE_ID = 2;
 export const ORGANISATION_OTHER_PRIVATE_INSURANCE_ID = 3;
-export const ORGANISATION_NO_COMPANY_ID = 31;
\ No newline at end of file
+export const ORGANISATION_NO_COMPANY_ID = 31;
diff --git a/client/src/pages/StyleGuidePage.vue b/client/src/pages/StyleGuidePage.vue
index 26f791bb..25ff5db5 100644
--- a/client/src/pages/StyleGuidePage.vue
+++ b/client/src/pages/StyleGuidePage.vue
@@ -13,7 +13,6 @@ import VerticalBarChart from "@/components/ui/VerticalBarChart.vue";
 import LearningPathCircle from "@/pages/learningPath/learningPathPage/LearningPathCircle.vue";
 import logger from "loglevel";
 import { reactive, ref } from "vue";
-import VueDatePicker from "@vuepic/vue-datepicker";
 import "@vuepic/vue-datepicker/dist/main.css";
 
 const state = reactive({
@@ -36,8 +35,6 @@ const state = reactive({
   },
 });
 
-const birtDate = ref("1982-06-15");
-
 const dropdownData = [
   {
     title: "Option 1",
@@ -415,11 +412,6 @@ function log(data: any) {
     ></ItDropdownSelect>
     {{ state.dropdownSelected }}
 
-    <h2 class="mb-8 mt-8">Date Picker</h2>
-    <div class="mt-2">
-      <VueDatePicker v-model="birthDate"></VueDatePicker>
-    </div>
-
     <h2 class="mb-8 mt-8">Checkbox</h2>
 
     <ItCheckbox
diff --git a/client/src/pages/onboarding/AccountProfile.vue b/client/src/pages/onboarding/AccountProfile.vue
index 2982a9a3..727dd96e 100644
--- a/client/src/pages/onboarding/AccountProfile.vue
+++ b/client/src/pages/onboarding/AccountProfile.vue
@@ -100,7 +100,7 @@ async function navigateNextRoute() {
       <ItDropdownSelect v-model="selectedOrganisation" :items="organisations" />
 
       <div v-if="isOtherOrganisation(selectedOrganisation.id)" class="my-8">
-        <label for="organisationDetailName" class="block pb-1.5 heading-3">
+        <label for="organisationDetailName" class="heading-3 block pb-1.5">
           {{ $t("a.Firmenname") }}
         </label>
         <input
diff --git a/client/src/pages/personalProfile/PersonalProfilePage.vue b/client/src/pages/personalProfile/PersonalProfilePage.vue
index 01903add..99312b07 100644
--- a/client/src/pages/personalProfile/PersonalProfilePage.vue
+++ b/client/src/pages/personalProfile/PersonalProfilePage.vue
@@ -45,7 +45,11 @@ function startEditMode() {
 
     <div class="flex flex-grow flex-col space-y-4 px-8 py-8 md:px-16">
       <div v-if="!editMode" class="flex justify-end space-x-4">
-        <button class="btn btn-secondary" @click="startEditMode">
+        <button
+          class="btn btn-secondary"
+          data-cy="editProfileButton"
+          @click="startEditMode"
+        >
           {{ $t("a.Profil bearbeiten") }}
         </button>
       </div>
diff --git a/cypress/e2e/checkout-vv/checkout.cy.js b/cypress/e2e/checkout-vv/checkout.cy.js
index 00f39ac7..e6007b44 100644
--- a/cypress/e2e/checkout-vv/checkout.cy.js
+++ b/cypress/e2e/checkout-vv/checkout.cy.js
@@ -17,22 +17,31 @@ describe("checkout.cy.js", () => {
 
     cy.get('[data-cy="account-confirm-title"]').should(
       "contain",
-      "Konto erstellen"
+      "Konto erstellen",
     );
     cy.get('[data-cy="continue-button"]').click();
 
     cy.get('[data-cy="account-profile-title"]').should(
       "contain",
-      "Profil ergänzen"
+      "Profil ergänzen",
     );
     cy.get('[data-cy="dropdown-select"]').click();
-    cy.get('[data-cy="dropdown-select-option-Baloise"]').click();
+    cy.get(
+      '[data-cy="dropdown-select-option-andere Krankenversicherer"]',
+    ).click();
+    cy.get("#organisationDetailName").type("FdH GmbH");
     cy.get('[data-cy="continue-button"]').click();
 
+    cy.loadUser("id", TEST_USER_EMPTY_ID).then((u) => {
+      expect(u.organisation_detail_name).to.equal("FdH GmbH");
+      // 2 -> andere Krankenversicherer
+      expect(u.organisation).to.equal(2);
+    });
+
     // Adressdaten ausfüllen
     cy.get('[data-cy="account-checkout-title"]').should(
       "contain",
-      "Lehrgang kaufen"
+      "Lehrgang kaufen",
     );
     cy.get("#street-address").type("Eggersmatt");
     cy.get("#street-number").type("32");
@@ -40,7 +49,7 @@ describe("checkout.cy.js", () => {
     cy.get("#city").type("Zumholz");
     cy.get('[data-cy="add-company-address"]').click();
 
-    cy.get("#company-name").type("Iterativ GmbH");
+    // cy.get("#company-name").clear().type("Iterativ GmbH");
     cy.get("#company-street-address").type("Brückfeldstrasse");
     cy.get("#company-street-number").type("16");
     cy.get("#company-postal-code").type("3012");
@@ -60,7 +69,7 @@ describe("checkout.cy.js", () => {
       expect(ci.country).to.equal("CH");
 
       expect(ci.invoice_address).to.equal("org");
-      expect(ci.organisation_detail_name).to.equal("Iterativ GmbH");
+      expect(ci.organisation_detail_name).to.equal("FdH GmbH");
       expect(ci.organisation_street).to.equal("Brückfeldstrasse");
       expect(ci.organisation_street_number).to.equal("16");
       expect(ci.organisation_postal_code).to.equal("3012");
@@ -72,12 +81,32 @@ describe("checkout.cy.js", () => {
       expect(ci.state).to.equal("ongoing");
     });
 
+    cy.loadUser("id", TEST_USER_EMPTY_ID).then((u) => {
+      expect(u.first_name).to.equal("Flasche");
+      expect(u.last_name).to.equal("Leer");
+
+      expect(u.street).to.equal("Eggersmatt");
+      expect(u.street_number).to.equal("32");
+      expect(u.postal_code).to.equal("1719");
+      expect(u.city).to.equal("Zumholz");
+      expect(u.country).to.equal("CH");
+
+      expect(u.invoice_address).to.equal("org");
+      expect(u.organisation_detail_name).to.equal("FdH GmbH");
+      expect(u.organisation_street).to.equal("Brückfeldstrasse");
+      expect(u.organisation_street_number).to.equal("16");
+      expect(u.organisation_postal_code).to.equal("3012");
+      expect(u.organisation_city).to.equal("Bern");
+      // 2 -> andere Krankenversicherer
+      expect(u.organisation).to.equal(2);
+    });
+
     // pay
     cy.get('[data-cy="pay-button"]').click();
 
     cy.get('[data-cy="checkout-success-title"]').should(
       "contain",
-      "Gratuliere"
+      "Gratuliere",
     );
     // wait for payment callback
     cy.wait(3000);
@@ -86,7 +115,7 @@ describe("checkout.cy.js", () => {
     // back on dashboard page
     cy.get('[data-cy="db-course-title"]').should(
       "contain",
-      "Versicherungsvermittler"
+      "Versicherungsvermittler",
     );
 
     cy.loadCheckoutInformation("user_id", TEST_USER_EMPTY_ID).then((ci) => {
@@ -102,13 +131,13 @@ describe("checkout.cy.js", () => {
 
     cy.get('[data-cy="account-confirm-title"]').should(
       "contain",
-      "Konto erstellen"
+      "Konto erstellen",
     );
     cy.get('[data-cy="continue-button"]').click();
 
     cy.get('[data-cy="account-profile-title"]').should(
       "contain",
-      "Profil ergänzen"
+      "Profil ergänzen",
     );
     cy.get('[data-cy="dropdown-select"]').click();
     cy.get('[data-cy="dropdown-select-option-Baloise"]').click();
@@ -117,10 +146,10 @@ describe("checkout.cy.js", () => {
     // Adressdaten ausfüllen
     cy.get('[data-cy="account-checkout-title"]').should(
       "contain",
-      "Lehrgang kaufen"
+      "Lehrgang kaufen",
     );
 
-    cy.get('#paymentMethod').select('cembra_byjuno');
+    cy.get("#paymentMethod").select("cembra_byjuno");
 
     cy.get("#street-address").type("Eggersmatt");
     cy.get("#street-number").type("32");
@@ -132,32 +161,38 @@ describe("checkout.cy.js", () => {
 
     cy.get('[data-cy="continue-pay"]').click();
 
-    cy.loadExternalApiRequestLog("request_username", "empty@example.com").then((entry) => {
-      // ends with "/v1/transactions""
-      expect(entry.api_url).to.contain("/v1/transactions");
-      expect(entry.request_username).to.contain("empty@example.com");
+    cy.loadExternalApiRequestLog("request_username", "empty@example.com").then(
+      (entry) => {
+        // ends with "/v1/transactions""
+        expect(entry.api_url).to.contain("/v1/transactions");
+        expect(entry.request_username).to.contain("empty@example.com");
 
-      expect(entry.api_request_data.amount).to.equal(32400);
-      expect(entry.api_request_data.currency).to.equal("CHF");
-      expect(entry.api_request_data.autoSettle).to.equal(true);
+        expect(entry.api_request_data.amount).to.equal(32400);
+        expect(entry.api_request_data.currency).to.equal("CHF");
+        expect(entry.api_request_data.autoSettle).to.equal(true);
 
-      expect(entry.api_request_data.customer.firstName).to.equal("Flasche");
-      expect(entry.api_request_data.customer.lastName).to.equal("Leer");
-      expect(entry.api_request_data.customer.street).to.equal("Eggersmatt 32");
-      expect(entry.api_request_data.customer.zipCode).to.equal("1719");
-      expect(entry.api_request_data.customer.city).to.equal("Zumholz");
-      expect(entry.api_request_data.customer.country).to.equal("CH");
-      expect(entry.api_request_data.customer.type).to.equal("P");
-      expect(entry.api_request_data.customer.phone).to.equal("+41792018586");
-      expect(entry.api_request_data.customer.birthDate).to.equal("1982-06-09");
+        expect(entry.api_request_data.customer.firstName).to.equal("Flasche");
+        expect(entry.api_request_data.customer.lastName).to.equal("Leer");
+        expect(entry.api_request_data.customer.street).to.equal(
+          "Eggersmatt 32",
+        );
+        expect(entry.api_request_data.customer.zipCode).to.equal("1719");
+        expect(entry.api_request_data.customer.city).to.equal("Zumholz");
+        expect(entry.api_request_data.customer.country).to.equal("CH");
+        expect(entry.api_request_data.customer.type).to.equal("P");
+        expect(entry.api_request_data.customer.phone).to.equal("+41792018586");
+        expect(entry.api_request_data.customer.birthDate).to.equal(
+          "1982-06-09",
+        );
 
-      expect(entry.api_request_data.INT.repaymentType).to.equal(3);
-      expect(entry.api_request_data.INT.riskOwner).to.equal("IJ");
-      expect(entry.api_request_data.INT.subtype).to.equal("INVOICE");
-      expect(entry.api_request_data.INT.deviceFingerprintId).to.not.be.empty;
+        expect(entry.api_request_data.INT.repaymentType).to.equal(3);
+        expect(entry.api_request_data.INT.riskOwner).to.equal("IJ");
+        expect(entry.api_request_data.INT.subtype).to.equal("INVOICE");
+        expect(entry.api_request_data.INT.deviceFingerprintId).to.not.be.empty;
 
-      expect(true).to.be.true;
-    });
+        expect(true).to.be.true;
+      },
+    );
 
     // check that results are stored on server
     cy.loadCheckoutInformation("user_id", TEST_USER_EMPTY_ID).then((ci) => {
@@ -183,5 +218,23 @@ describe("checkout.cy.js", () => {
       expect(ci.ip_address).to.not.be.empty;
       expect(ci.device_fingerprint_session_key).to.not.be.empty;
     });
+
+    cy.loadUser("id", TEST_USER_EMPTY_ID).then((u) => {
+      expect(u.first_name).to.equal("Flasche");
+      expect(u.last_name).to.equal("Leer");
+
+      expect(u.street).to.equal("Eggersmatt");
+      expect(u.street_number).to.equal("32");
+      expect(u.postal_code).to.equal("1719");
+      expect(u.city).to.equal("Zumholz");
+      expect(u.country).to.equal("CH");
+      expect(u.phone_number).to.equal("+41792018586");
+      expect(u.birth_date).to.equal("1982-06-09");
+
+      expect(u.invoice_address).to.equal("prv");
+
+      // 7 -> Baloise
+      expect(u.organisation).to.equal(7);
+    });
   });
 });
diff --git a/cypress/e2e/profile/profile.cy.js b/cypress/e2e/profile/profile.cy.js
new file mode 100644
index 00000000..627e8088
--- /dev/null
+++ b/cypress/e2e/profile/profile.cy.js
@@ -0,0 +1,85 @@
+import { TEST_USER_EMPTY_ID } from "../../consts";
+import { login } from "../helpers";
+
+describe("profile.cy.js", () => {
+  beforeEach(() => {
+    cy.manageCommand("cypress_reset");
+
+    login("empty@example.com", "test");
+    cy.visit("/profile");
+  });
+
+  it("can edit all profile fields", () => {
+    cy.get('[data-cy="editProfileButton"]').click();
+
+    cy.get("#phone").type("079 201 85 86");
+    cy.get('[data-test="dp-input"]').type("09.06.1982{enter}");
+
+    cy.get("#street-address").type("Hafen");
+    cy.get("#street-number").type("123");
+    cy.get("#postal-code").type("DE-20095");
+    cy.get("#city").type("Hamburg");
+    cy.get("#country").select("DE");
+
+    // andere broker
+    cy.get("#organisation").select("1");
+
+    cy.get("#org-detail-name").type("Judihui GmbH");
+    cy.get("#org-street-address").type("Auf der Alm");
+    cy.get("#org-street-number").type("17");
+    cy.get("#org-postal-code").type("AT-6020");
+    cy.get("#org-city").type("Innsbruck");
+    cy.get("#org-country").select("AT");
+
+    cy.get("#invoice-address-organisation").click();
+
+    cy.get('[data-cy="saveButton"]').click();
+
+    // check displayed data
+    cy.get('[data-cy="firstName"]').should("contain", "Flasche");
+    cy.get('[data-cy="lastName"]').should("contain", "Leer");
+    cy.get('[data-cy="email"]').should("contain", "empty@example.com");
+    cy.get('[data-cy="phone"]').should("contain", "079 201 85 86");
+    cy.get('[data-cy="birthDate"]').should("contain", "09.06.1982");
+
+    cy.get('[data-cy="privateAddress"]').should("contain", "Hafen 123");
+    cy.get('[data-cy="privateAddress"]').should("contain", "DE-20095 Hamburg");
+    cy.get('[data-cy="privateAddress"]').should("contain", "Deutschland");
+
+    cy.get('[data-cy="organisationDetailName"]').should(
+      "contain",
+      "andere Broker",
+    );
+    cy.get('[data-cy="organisationAddress"]').should("contain", "Judihui GmbH");
+    cy.get('[data-cy="organisationAddress"]').should(
+      "contain",
+      "Auf der Alm 17",
+    );
+    cy.get('[data-cy="organisationAddress"]').should(
+      "contain",
+      "AT-6020 Innsbruck",
+    );
+    cy.get('[data-cy="organisationAddress"]').should("contain", "Österreich");
+
+    // check stored data
+    cy.loadUser("id", TEST_USER_EMPTY_ID).then((u) => {
+      expect(u.first_name).to.equal("Flasche");
+      expect(u.last_name).to.equal("Leer");
+
+      expect(u.street).to.equal("Hafen");
+      expect(u.street_number).to.equal("123");
+      expect(u.postal_code).to.equal("DE-20095");
+      expect(u.city).to.equal("Hamburg");
+      expect(u.country).to.equal("DE");
+
+      expect(u.invoice_address).to.equal("org");
+      expect(u.organisation_detail_name).to.equal("Judihui GmbH");
+      expect(u.organisation_street).to.equal("Auf der Alm");
+      expect(u.organisation_street_number).to.equal("17");
+      expect(u.organisation_postal_code).to.equal("AT-6020");
+      expect(u.organisation_city).to.equal("Innsbruck");
+      // 1 -> andere Broker
+      expect(u.organisation).to.equal(1);
+    });
+  });
+});
diff --git a/cypress/support/commands.js b/cypress/support/commands.js
index 50dfdf30..3564a74c 100644
--- a/cypress/support/commands.js
+++ b/cypress/support/commands.js
@@ -178,6 +178,17 @@ Cypress.Commands.add("loadCheckoutInformation", (key, value) => {
   );
 });
 
+Cypress.Commands.add("loadUser", (key, value) => {
+  return loadObjectJson(
+    key,
+    value,
+    "vbv_lernwelt.core.models.User",
+    "vbv_lernwelt.core.serializers.CypressUserSerializer",
+    true
+  );
+});
+
+
 Cypress.Commands.add("makeSelfEvaluation", (answers) => {
   for (let i = 0; i < answers.length; i++) {
     const answer = answers[i];
diff --git a/server/vbv_lernwelt/core/serializers.py b/server/vbv_lernwelt/core/serializers.py
index 3a656c71..31965017 100644
--- a/server/vbv_lernwelt/core/serializers.py
+++ b/server/vbv_lernwelt/core/serializers.py
@@ -131,6 +131,12 @@ class UserSerializer(serializers.ModelSerializer):
         return instance
 
 
+class CypressUserSerializer(serializers.ModelSerializer):
+    class Meta:
+        model = User
+        fields = "__all__"
+
+
 class OrganisationSerializer(serializers.ModelSerializer):
     id = serializers.IntegerField(source="organisation_id", read_only=True)
     name = serializers.SerializerMethodField()

From 8882e04049e17bd9e36e9cd8f3f3de7a5aef52f7 Mon Sep 17 00:00:00 2001
From: Daniel Egger <daniel.egger@gmail.com>
Date: Tue, 16 Jul 2024 16:43:20 +0200
Subject: [PATCH 05/12] Rename cypress test to `personaProfile`

---
 .../profile.cy.js => personalProfile/personalProfile.cy.js}     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename cypress/e2e/{profile/profile.cy.js => personalProfile/personalProfile.cy.js} (98%)

diff --git a/cypress/e2e/profile/profile.cy.js b/cypress/e2e/personalProfile/personalProfile.cy.js
similarity index 98%
rename from cypress/e2e/profile/profile.cy.js
rename to cypress/e2e/personalProfile/personalProfile.cy.js
index 627e8088..2608f53b 100644
--- a/cypress/e2e/profile/profile.cy.js
+++ b/cypress/e2e/personalProfile/personalProfile.cy.js
@@ -1,7 +1,7 @@
 import { TEST_USER_EMPTY_ID } from "../../consts";
 import { login } from "../helpers";
 
-describe("profile.cy.js", () => {
+describe("personalProfile.cy.js", () => {
   beforeEach(() => {
     cy.manageCommand("cypress_reset");
 

From 96e9ef63574c2fd530015468649dc8e92cb0b8f8 Mon Sep 17 00:00:00 2001
From: Daniel Egger <daniel.egger@gmail.com>
Date: Tue, 16 Jul 2024 16:47:19 +0200
Subject: [PATCH 06/12] Add third cypress/currents step

---
 bitbucket-pipelines.yml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/bitbucket-pipelines.yml b/bitbucket-pipelines.yml
index 909e9bdb..0dcb4cc1 100644
--- a/bitbucket-pipelines.yml
+++ b/bitbucket-pipelines.yml
@@ -97,6 +97,7 @@ js-linting: &js-linting
 
 default-steps: &default-steps
   - parallel:
+      - step: *e2e
       - step: *e2e
       - step: *e2e
       - step: *python-tests

From 0fef8ec9b693b410102940e1fb0acaa40f7b5382 Mon Sep 17 00:00:00 2001
From: Christian Cueni <christian.cueni@iterativ.ch>
Date: Tue, 23 Jul 2024 13:55:16 +0200
Subject: [PATCH 07/12] Extend evaluation period

---
 server/vbv_lernwelt/importer/services.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/server/vbv_lernwelt/importer/services.py b/server/vbv_lernwelt/importer/services.py
index f4594a24..20de544f 100644
--- a/server/vbv_lernwelt/importer/services.py
+++ b/server/vbv_lernwelt/importer/services.py
@@ -765,7 +765,7 @@ def create_or_update_course_session_assignment(
             csa.submission_deadline.save()
             csa.evaluation_deadline.start = timezone.make_aware(
                 start
-            ) + timezone.timedelta(days=45)
+            ) + timezone.timedelta(days=60)
             csa.evaluation_deadline.end = None
             csa.evaluation_deadline.save()
     else:

From 46daea051153112fcb2b1290004e42fd5849c10a Mon Sep 17 00:00:00 2001
From: Daniel Egger <daniel.egger@gmail.com>
Date: Thu, 18 Jul 2024 14:19:34 +0200
Subject: [PATCH 08/12] Enable cembra/byjuno payment

---
 env_secrets/caprover_vbv-develop.env          | Bin 1410 -> 1410 bytes
 server/vbv_lernwelt/core/constants.py         |   1 +
 .../vbv_lernwelt/core/create_default_users.py |  19 ++++++++++++++++++
 server/vbv_lernwelt/shop/services.py          |   6 ++----
 4 files changed, 22 insertions(+), 4 deletions(-)

diff --git a/env_secrets/caprover_vbv-develop.env b/env_secrets/caprover_vbv-develop.env
index c633f417bff7fbf9e1da651afdf0215ac04467d3..5432d3c8d0f3325116f2c115d5b3e00b05d3e2f7 100644
GIT binary patch
literal 1410
zcmV-|1%3JeM@dveQdv+`0R5m0PB9m#27BrtQzTx)WmcPXSslete|WanHJO_H|4~_O
zP^cLG*XtL4O<!-W+xD2=#uhe|e1V<mhp%nf?u?wUD_cxP)i(Irj&{N%Q*tRMD)8w;
zZ=>C<rX6cOlJ7-?>~z@$5m_&IGL&h_JQJ|gH+5hQSp4{>H?-RSNu(A%LMc|^)u)D&
zqoMe<u77mXu6p2oq9q(ZJYwI}AL#j!M!KJK&gMsmEf$}d#To%CozsdWhd(C6E!#*t
z(g18RFC-UJ0}q)#l%qzS+2*&qmTKo({?g>#h&R6`nYgW1niNZQJRV!nlGJJM71d4a
zXb5fj0W&mrii<m~CrMWp${t3*W0OVqo=ZBOM`nnh-YA71P8%0jdZD`tOia4t{Fl*G
zC@2Tfv+#;*OnsTg3|)%A={t{ys%{}#OhApxPZ5KP<~+T-m2%Z%oSm!-JT&Fa>@$Dr
zb6~$Ch`Cn`pm{iLtmmv{cu9*`reNXYG0EPEm>5}eK2Jx!xd-5<WGv@zo#$hBHlEpV
z93Y`gp&_R~xcWo&nEYqfjfMJL$Y^!wBCU6Ej|YG%3h_CJFAD>2RUe}Z{XQJ;rN>r~
zQTuqp<1^BI105)RWpaFZn1K(pE4kPlby`GZL5+BR5HnnU8lO_ok{Hz>+V||0fO%&>
zUTzCfno(qE>jU{9E`Lqi4WZjF-*@p<N}(<>gQQlH2*r>PZRedmIMZ09JE%OVhuP`f
z9(|?;L{^b-XRf`qCIYFZze%{^2W&(Mp#mub)onH8O#d{_r-3NY+RWFWVHCV2fbg3B
z)v$P*s2Bz3EaT`je|}Py(UWG)=0tl=$KTk}n~-5=lr(V3!gFD|T@OLQoY#?+@5px+
zcV2^PdKzL=TJ#>u{aaMN2S&|{kdKeyJ7M~ufHh8^AnKB|jS*DCDPLi<&FsYaRh@S~
zzbM3%GT4NC7#wuTUf+nuWoFAO%iOBfPou;>b_tOj>nBsX6}Q}PN%-*<=T!5Fez2>A
zw|xDViOuevU9QfXWEV0qcOKn2R?;&S-|3^2tv&0H#uBTB@bM)R<WTeV(+}&z|6uC~
zn+%^n2Y#+wlDW>i=A8FMH(+HRE6*f?OQWX8su^Ien`@qnd_dom`|@eKUW(Q$#e&BT
z#De4lBb$-MEwE)<Sk|L-54OP&AIO6Y2dml=iF-K$l`rh%n{4@bT@VsU#mvFz3s;A0
zy<Yz1DjiL|M^+r=Wv#Yv%0LKv%yE@{<6?#^aObhTk-}xDu{ISgWYfKFeYhFSG|P<+
zuWR+AwINJ{HAiN<w!mdzR>l9U!rIU0>zkK$LoZUulYSL!ZT8*JM##eJNbm;78K|~f
zbB>Z`XP`R`>Sry?a1a*IAEugrmH`9IZIS&zP}DXhzGNu67xQ>!Do?$tiz^ICk-nh}
z2Oo<H=sb=5d^m_!)sz%|e5ArpFGXH+3ga<U@eq1dXpZF^KEEx53d%5ic7mi}(X13%
zm1SPgABIhK-wdv0+=~2a-3r+(THah2Bfp*l0nuN=ht^1clx2<P!Zwsx*zBL4e4!84
z8qc+}Ysq_qnXcNh8&o2t{Oew6=@=mvmF&1hcaL;7y7cLyxC?2{NWZvDew(EhnRtc1
zJozgL<&iU=n6mPh=@^r_hsOS?6WJG;G2gL3ysr|WO)0y*`(_=S%n~jEUP2;G-sW~Z
zk>(KyVR(yT*ysaU>N#&P^6;*_#1ww}``u5gjWb7s%&EwLsry38dxw5(Wo2x?h4oy3
z&M+{+21EE%bNplpX5mto&aIU&7gB2cEC5LNl)jY^=~4hooIYAOM;UY&L3$bkOxgx&
QEL5k|q77$mx!byyB&9#T9{>OV

literal 1410
zcmV-|1%3JeM@dveQdv+`0H6*;YgY}<6t)6kQ+BGQ30$%8=Z<g;h43(^^9+i;bm1yK
zXqsCdGk+VrAtg(<)2?~&1~PHGC#jShY)!{nP5kwfQoU1Ym5BF(`!vuD{cRy^v@N!c
zSU!!v1G~2X)NWBI<<=dMem!%J3<)ihe~`q9=GlIW>&H;LwH8|#dM(Qy-L0^4mFL{A
z{NOuVz~v=y&_o7Y2`$$o!*}E<`7vnJ!af<$q!0r!FoM@Q#HAMc7Y&puEGvnhUK!Di
z&>2kCU#QHjSZ*)RWr}W_-ndPAs6A9{B-bqRRv1_d$1ci^n2h{)voEKLu@-BPHHgj=
zIcR*n@Gc5NE4JXOuR{(7cA#3$7&WCYHL|4UuzX<%h`hCe2C#wII#_HxS`rIm{T59>
zQIqlroBTHL_d;@R67in5rPfhH=hP||Vx0)R2jyK6QaJis6y(d`{|zqSK?^wz$tQpk
z*Ln%bPVBy0MRsqd^E^AV;>Ck45A054(u2={U!U*eAim-{{i!8P^4r$k9zGo;=txaC
zbk8H+VA`T(h%-(QoxunK%<5euAlxhd*RHby45xd8Na&=o8V&bp6Ie=13`(wG--U3U
z%6uo>T?8;FCcrtkuL_SoNb<!rO&h@i@xGNkIhK83dit+pV0Qx4pP!`8wZBBvMzDCg
z!ORQA1i;>^C`C11abt?$R$rxb5)Z%Rx0~jI)abor+2+mP?$a0$`>QlwukUn(cQ5@4
zxGGGP^*jr0%RhNrwF3ULY8pz?Q3q1)+YjQav-_RqC<DYGpn*U};We>|zkG_*1ZK0!
z%8dr2?!i9*Zra&A(nkG8Ab&+uW>B>f)4;am&HjfY%OFi;A@(`zq*q>dDN#sP<U<mN
zWwH|yp&ra2hQ%B_TFGc~bPt*GJ=laJlHMW11!5lu2d^DZ3uKW+@><(&CR`F@$$&20
z3@e~YOhE1In9IS{w8!hest|~(A;56ZAbq9}ujdpnJ6S~TH=OS(d=FmzF;6B`ls!I#
z87_xkLl;Ui7k4~2DddBU2}V}r2i#fO@-L56NJ;dWt>3`M1?zebcvO@Uj1a(k2ygSC
zIVrx#$;S>X`*z9m+}Xei1Igz|#i!eA`<l0!or1W%w=6$4sbc79iK<FeQK9HJdj=I^
z?{S=;pXwgi_X~Td4MYUt7q4+oFYy;8{ClmvPkos8hD2P^JaUa_%(u#v*?)%Y)f=H)
z|9K<*WLUmdx?l+<=h()Yq$1K1dYQ{Cq(zZ^s!We%C+KA2hB55^&l?{BbDGxh=H%J1
z&koOM7rxrrLV0-ak3KIG#CiV*lz;bhPyU@;T}&wyvhKEu+{DD5HTMTfbg<TECHmu=
zcISXLF4Q!}5KeG15Jf+tVcm^332Qm4Yel1OM3uA>!`r6|8)RF2YG?nE)HXHvZT+b>
zk6{cxE^)~X#Al4Ih%`RZf3P=8sEjg+Xu+1po@o8eSVb)V)I<Gh{d}}lE4%s2ZiHGG
zgHxTFV3NSLO*oQyp<K3+@hOdKPPsEz>6n$9O2`_ZG5ED;ZXFH)nK+YnTHlca;&1`%
z0jx<iI30WU?N%Nm(3sb*vF?})h&N4coV)8@aMRzTr6dkp)*l1)9()=o)U>g<@ZXhY
zUpx6u^|SdDbcl|L(}T!Zq|99VcAqN4frV+d6ES4u3j7*nR)2AN1ohg<!hlV`j-pYd
zHMN}L1G5lcNN0km%hrb6J+xz_<5H*u)bD6A4A`%4h)|6UEUAgzvmW#te?5@Pjp89`
zG|D;pIy1Qv8=80H7N2;V*A)17&<cRN_&cCSHmnew{+j>j>um4M;K-H0d!IS0W+DTr
QcJP3A1@w{)_wTYYEKec0#Q*>R

diff --git a/server/vbv_lernwelt/core/constants.py b/server/vbv_lernwelt/core/constants.py
index 75f24848..e36702b8 100644
--- a/server/vbv_lernwelt/core/constants.py
+++ b/server/vbv_lernwelt/core/constants.py
@@ -28,6 +28,7 @@ TEST_MENTOR1_USER_ID = "d1f5f5a9-5b0a-4e1a-9e1a-9e9b5b5e1b1b"
 TEST_STUDENT1_VV_USER_ID = "5ff59857-8de5-415e-a387-4449f9a0337a"
 TEST_STUDENT2_VV_AND_VV_MENTOR_USER_ID = "7e8ebf0b-e6e2-4022-88f4-6e663ba0a9db"
 TEST_USER_EMPTY_ID = "daecbabe-4ab9-4edf-a71f-4119042ccb02"
+TEST_USER_DATATRANS_HANNA_ID = "6bec1a0d-f852-47aa-a4de-072df6e07ad1"
 
 TEST_COURSE_SESSION_BERN_ID = -1
 TEST_COURSE_SESSION_ZURICH_ID = -2
diff --git a/server/vbv_lernwelt/core/create_default_users.py b/server/vbv_lernwelt/core/create_default_users.py
index 17acd036..a384c13e 100644
--- a/server/vbv_lernwelt/core/create_default_users.py
+++ b/server/vbv_lernwelt/core/create_default_users.py
@@ -20,6 +20,7 @@ from vbv_lernwelt.core.constants import (
     TEST_SUPERVISOR1_USER_ID,
     TEST_TRAINER1_USER_ID,
     TEST_TRAINER2_USER_ID,
+    TEST_USER_DATATRANS_HANNA_ID,
     TEST_USER_EMPTY_ID,
 )
 from vbv_lernwelt.core.models import User
@@ -202,6 +203,24 @@ def create_default_users(default_password="test", set_avatar=False):
         language="de",
     )
 
+    hanna, _ = User.objects.get_or_create(
+        id=TEST_USER_DATATRANS_HANNA_ID,
+    )
+    hanna.username = "datatrans.hanna.vbv@example.com"
+    hanna.email = "datatrans.hanna.vbv@example.com"
+    hanna.language = "de"
+    hanna.first_name = "Hanna"
+    hanna.last_name = "Vbv"
+    hanna.street = "Bahnstrasse"
+    hanna.street_number = "2"
+    hanna.postal_code = "8603"
+    hanna.city = "Schwerzenbach"
+    hanna.country_id = "CH"
+    hanna.birth_date = "1970-01-01"
+    hanna.phone_number = "+41792018586"
+    hanna.password = make_password("test")
+    hanna.save()
+
     for user_data in default_users:
         _create_student_user(**user_data)
 
diff --git a/server/vbv_lernwelt/shop/services.py b/server/vbv_lernwelt/shop/services.py
index f5998d38..a9561d0e 100644
--- a/server/vbv_lernwelt/shop/services.py
+++ b/server/vbv_lernwelt/shop/services.py
@@ -7,7 +7,6 @@ import structlog
 from django.conf import settings
 
 from vbv_lernwelt.core.admin import User
-from vbv_lernwelt.shop.const import VV_PRODUCT_NUMBER
 from vbv_lernwelt.shop.datatrans.datatrans_api_client import DatatransApiClient
 from vbv_lernwelt.shop.models import CheckoutState
 
@@ -101,9 +100,8 @@ def init_datatrans_transaction(
         },
     }
 
-    # FIXME: test with working cembra byjuno invoice customer?
-    # if with_cembra_byjuno_invoice:
-    #     payload["paymentMethods"] = ["INT"]
+    if with_cembra_byjuno_invoice:
+        payload["paymentMethods"] = ["INT"]
     if datatrans_customer_data:
         payload["customer"] = datatrans_customer_data
     if datatrans_int_data:

From e803e5308fc6bc948932901c92667b28c81ef513 Mon Sep 17 00:00:00 2001
From: Daniel Egger <daniel.egger@gmail.com>
Date: Thu, 18 Jul 2024 14:35:41 +0200
Subject: [PATCH 09/12] Fix unit tests

---
 .../vbv_lernwelt/core/create_default_users.py | 42 +++++++++++--------
 .../core/management/commands/cypress_reset.py |  5 +++
 2 files changed, 30 insertions(+), 17 deletions(-)

diff --git a/server/vbv_lernwelt/core/create_default_users.py b/server/vbv_lernwelt/core/create_default_users.py
index a384c13e..beeaca53 100644
--- a/server/vbv_lernwelt/core/create_default_users.py
+++ b/server/vbv_lernwelt/core/create_default_users.py
@@ -4,6 +4,7 @@ from django.contrib.auth.models import Group, Permission
 from django.core.files import File
 from environs import Env
 
+from vbv_lernwelt.core.model_utils import add_countries
 from vbv_lernwelt.media_files.models import UserImage
 
 env = Env()
@@ -79,7 +80,30 @@ default_users = [
 AVATAR_DIR = settings.APPS_DIR / "static" / "avatars"
 
 
+def create_datatrans_hanna_user():
+    hanna, _ = User.objects.get_or_create(
+        id=TEST_USER_DATATRANS_HANNA_ID,
+    )
+    hanna.username = "datatrans.hanna.vbv@example.com"
+    hanna.email = "datatrans.hanna.vbv@example.com"
+    hanna.language = "de"
+    hanna.first_name = "Hanna"
+    hanna.last_name = "Vbv"
+    hanna.street = "Bahnstrasse"
+    hanna.street_number = "2"
+    hanna.postal_code = "8603"
+    hanna.city = "Schwerzenbach"
+    hanna.country_id = "CH"
+    hanna.birth_date = "1970-01-01"
+    hanna.phone_number = "+41792018586"
+    hanna.password = make_password("test")
+    hanna.save()
+    return hanna
+
+
 def create_default_users(default_password="test", set_avatar=False):
+    add_countries(small_set=True)
+
     admin_group, created = Group.objects.get_or_create(name="admin_group")
     _content_creator_group, _created = Group.objects.get_or_create(
         name="content_creator_grop"
@@ -203,23 +227,7 @@ def create_default_users(default_password="test", set_avatar=False):
         language="de",
     )
 
-    hanna, _ = User.objects.get_or_create(
-        id=TEST_USER_DATATRANS_HANNA_ID,
-    )
-    hanna.username = "datatrans.hanna.vbv@example.com"
-    hanna.email = "datatrans.hanna.vbv@example.com"
-    hanna.language = "de"
-    hanna.first_name = "Hanna"
-    hanna.last_name = "Vbv"
-    hanna.street = "Bahnstrasse"
-    hanna.street_number = "2"
-    hanna.postal_code = "8603"
-    hanna.city = "Schwerzenbach"
-    hanna.country_id = "CH"
-    hanna.birth_date = "1970-01-01"
-    hanna.phone_number = "+41792018586"
-    hanna.password = make_password("test")
-    hanna.save()
+    hanna = create_datatrans_hanna_user()
 
     for user_data in default_users:
         _create_student_user(**user_data)
diff --git a/server/vbv_lernwelt/core/management/commands/cypress_reset.py b/server/vbv_lernwelt/core/management/commands/cypress_reset.py
index abfae404..1a02a343 100644
--- a/server/vbv_lernwelt/core/management/commands/cypress_reset.py
+++ b/server/vbv_lernwelt/core/management/commands/cypress_reset.py
@@ -17,8 +17,10 @@ from vbv_lernwelt.core.constants import (
     TEST_STUDENT2_VV_AND_VV_MENTOR_USER_ID,
     TEST_STUDENT3_USER_ID,
     TEST_TRAINER1_USER_ID,
+    TEST_USER_DATATRANS_HANNA_ID,
     TEST_USER_EMPTY_ID,
 )
+from vbv_lernwelt.core.create_default_users import create_datatrans_hanna_user
 from vbv_lernwelt.core.models import Organisation, User
 from vbv_lernwelt.course.consts import (
     COURSE_TEST_ID,
@@ -159,6 +161,9 @@ def command(
         password=make_password("test"),
     )
 
+    User.objects.filter(id=TEST_USER_DATATRANS_HANNA_ID).delete()
+    create_datatrans_hanna_user()
+
     cursor = connection.cursor()
     cursor.execute("truncate core_securityrequestresponselog;")
     cursor.execute("truncate core_externalapirequestlog;")

From 74c4099b8b3dd25bc494c26ba7ead13b1722f996 Mon Sep 17 00:00:00 2001
From: Daniel Egger <daniel.egger@gmail.com>
Date: Thu, 18 Jul 2024 15:29:12 +0200
Subject: [PATCH 10/12] Test-Deployment improvements

---
 caprover_create_app.py         |   8 ++++++--
 env_secrets/local_daniel.env   | Bin 1281 -> 1512 bytes
 server/config/settings/base.py |   6 ++++--
 3 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/caprover_create_app.py b/caprover_create_app.py
index 24dba123..e137bcf1 100644
--- a/caprover_create_app.py
+++ b/caprover_create_app.py
@@ -93,8 +93,12 @@ def main(app_name, image_name, environment_file):
             "AWS_S3_SECRET_ACCESS_KEY": env.str("AWS_S3_SECRET_ACCESS_KEY", ""),
             "AWS_S3_REGION_NAME": "eu-central-1",
             "AWS_STORAGE_BUCKET_NAME": "myvbv-dev.iterativ.ch",
-            "DATATRANS_HMAC_KEY": env.str("DATATRANS_HMAC_KEY", ""),
-            "DATATRANS_BASIC_AUTH_KEY": env.str("DATATRANS_BASIC_AUTH_KEY", ""),
+            "DATATRANS_HMAC_KEY": env.str("PIPELINES_DATATRANS_HMAC_KEY", ""),
+            "DATATRANS_BASIC_AUTH_KEY": env.str(
+                "PIPELINES_DATATRANS_BASIC_AUTH_KEY", ""
+            ),
+            "DATATRANS_API_ENDPOINT": "https://api.sandbox.datatrans.com",
+            "DATATRANS_PAY_URL": "https://pay.sandbox.datatrans.com",
             "FILE_UPLOAD_STORAGE": "s3",
             "IT_DJANGO_DEBUG": "false",
             "IT_SERVE_VUE": "false",
diff --git a/env_secrets/local_daniel.env b/env_secrets/local_daniel.env
index 7ae4f656c72a7fda433d8e81db09bbca0a48052d..703bd5e90348bdc749c714a4a40b8b13b4a5f3fb 100644
GIT binary patch
literal 1512
zcmV<E1sD1NM@dveQdv+`05m|e=o>C-quY~NB@>Kjv9hwx!L<9@4IcztOcYMjf4dlr
zN{=(4ZmZ=LJHWAVZg_dR8SFER-T^A(E;wTNd_Ffaoqjxx6dA~Bk<FQLncRa9U+7yw
zV8?Hn@Bc1dd761g{$uVn{(W5uu9t;#non!4xB1+wO`h309Dxd_K&XSCsl)X8#9&7j
zD-kKv(yg;Sk?m}iA&Fl&!hjrDV!DqwzLCUm%Mwi~ww$}uI{1AXRCoTK5gi59fzi<m
z$zmo_l))9X)h54lg0ajzmHaAO2~f07)MQNkZu}O`8_mdBIaZ`upJ57It!y|G980LR
zg`{2O1)BZucCE2o@C{Y`OeAov=RV)xFD)-(XhqP0cm!&#c+~S{%EIDLbnXQK6F}=+
zXh2Prpjq0(U!w;+fdoTLiOgD^_QPrbe|Iz<CDoNe=O_*)g-1P}hm*=L4yb}b>VTKf
zlDptT5_GKrzV^HV*6xpqTef^(SKk1azu@5Y=)vc$qN7^Qfhr2&q|tt)f8;+-*kRnZ
zCi#e@ID~vZZq{ZtYbjuRPa};}^4{Nu2h<Shx><y*%*yZeOp|QNtUw>dJl#GK+Rk29
zmXQUTC#j&e=T)}L+&hx-O^+1%oNDlhsa#ZzmfkV^rDKxArU`3r#3(qgL!Nef?>PU(
zz;EOB8BsWUX9ioh(kY1{3ZTWjWJ!5)U5p7PunF431sD?PnCB2rrg<?%fUyg+(p7M$
z(p-d3qRn7v+xFf0JI=dItC;s$jR!U#nN2vF6|{~!An1Z95*hExVWJ2gPQ*g|?895j
zl*-9T)kh$B@u9f30*aOx?ZS@|Nyd)4VAVJbUJ0O~1TSjkam3hwBqD~MPv=6<d#Efa
zWzi_&?7h^S9NCa+d9^B@c}`aQ?#A{TI?|h(G!Ey2s;P~D&^4tE@YQp8bi`+v$JSz!
z)tN(u=Z49ZBhMUvXLR}rPq3XeMkDl<GW&Hbv0?VUr!3tLXHnS?hMqeC8%ebpR@KhM
z0=~w!7LWw{dRJQ;UYEuBKewjNx--q|9Gfu4h^_ij%8sc>eCkGMCNN^$;u@3WS?GSE
zWPjq5k=b?&LI7L_9D>x~woW(*o47LD_5fH!WAkthP0hh6-L$PXy8fLU4W+;g2JO<I
zB(CgsS0-sRG`b2fTkUc;uU8~a6nvVZGP61YcTKz%__H4dRyyutknV-0uAp=&O-_x-
zZctQn4Jyc*wTq7n3ydDn>5aTvX$W&n@l!KbzgjdqI0skZC~XHbFz&!>OA1D{nRWDH
z=*?R+sETm?u)6H!xf?>X*z6-eT6jy~W(hVO9NU_<)f*^p%<CJt(yhF#3rwW{0L-oT
ztWvL_-qNJrf57(AvqN{MR>^{(aCXWZgCMGFeBYur-DF#>Z|#fVPZzSze8e<^AOz7h
zDhue^YZ>bCt8K*ywu;II74)@HlX8G>>$?S%Qsd`n?dkh+N{jfgB6jw!Q*T>WH1jdN
zM7?Dd*%d4Gp+jx|vYw69X64f}<MjxMWYIZ8MF)22@gJzhccu5<H^`-)C+u8PK-<Iq
ztOc6CZ-%a#L4?rcrWA~v4;R}VMp|E90?2LnR$$^EfrJD(fynpZXJqCrAx+z6Z}K1c
zX3^G+)5l2Qv>CKbx#qp{WTL-CD1ttmkP2rEI|K!5SyR3tpKuc-qW#&(Adey5U&$9^
zRw)ZhN``&1dSqd9n4s9=i;Eb0<2qK3Q#U4dfG6=0!Tqh@=9X#w)CiUDTDABYoPZOH
zq27Nv=l=sOWe5~s<XULKaX=448`@&R#5!LufbuIxVyppKz*Jg1m4u)>nBc(n5^PAO
zBF>DO`i))}nopj_CGh-b@b{}#Hkj77l(Ljr14+SP5qsnUfD5^EJ4w^DD!G>4A+a2=
zY+^R0N)P>FMjOrsLihd7Pwl_)le$doDXh%}J}a|Jcahl6fO8Nfq3{}Mj;UFAt7!OU
O%0;Y)orw|(_Ix()nD*WP

literal 1281
zcmV+c1^)T~M@dveQdv+`0K~zt1aShaqph1>SC&gOXbPlOPt<_H_XF=zFMrBbRB&HK
z>zIo@$>_iMEgH;_Jj`zfn7Q}W)m%2RD@&QnOlSeUhGQ^Ie&karq-#%x2ArcCTcMye
zacXtbU;>n5<LHE087#GoYw=3r<C$A+XesalkLztTzY)0SL8no7+C3&ANTO`{HB4kZ
zc;}UA`K2-DoZA9noyU+Do%LzSc*PK*kHg`kHs{z(KLidl^k|VJl~2(k{EaNuJ8q|!
zQuz&{%QjGoG88E`lm%Sl!J##^Sn>v|($s-;&(Jfw)#UCp0<qWez__?D&*s8_z8xRI
zysRtff^W{XQ6C|A1uPA?u|ORB+fXH_1;FZmg15K89uqv^^BQZg#awtof<=46Vi^C~
zxKQtZN|izUX@f>(*k!b2*IepL9U)KT4ok{a(-lNo^<zBvbMj7{g%(!!_KxRSOQ1!N
zt_&>u>?9P{CBUfv{I0GWg3vdtjSuJWUSy!9g6La*G)ut$7C8A-A-jfV@Sh<32q9E^
z^aSA0{3U^xSj4G$P$J}d-j0dX+895O2$b(tW^u@#rM+SPG;nnMI7;lfZZn)&S+7*$
zhPCb4wn**kjS>3mN-m3g+K=Y4HX8I&Yb#sAtB<yIZ6tp5{G0C;Z^<|$5}u15rWAY|
z3uu3$em*CoMC6lpAK8|)LeqD&OTB{Hv{WF8_LDNGjzF&vnP+~DJdAwbi%_t<DE&cR
z%{V=<8r#MxoQ!8XeIG4xi~wMd&cjb{#akC6{~;U6v+a9RPFi%EyeKzHWP_mxFsopi
zE==Ytz|s<*D13Xwt9uSE*)P!N88YN!b+ts7xqMo68{M6$bF{{1*S~Lx3goQdI};Xy
zPKx~`?33&-qp38^+FRs#QT8I^A$9@BiD>l=8Izg%+s;X(L(g(7+wI7Spvf#p8(@E+
zA^+tv>NM|Nx2-8=)l0f?;3OwesAAcBR&M5NhEE$jS&MZNw3-7=Kg<G;Da(PBVlH!&
zZI%9!s$k$odfQ|OJw&dS*eaVgp5G(a9-=EI)2!>RZ4WwVD5C%ws150Btv7VWX<v6A
zVuF||roNdG8*3)dIv=-VTfW7TVFn4&-E$*5Vom>E)*qrxAX<49+iiA!d({XH8<HAp
zph3;RG!nFRkYnC)%H~7oE{4TnS9`7|c%)i{M#GoDn{~F<RRvW7gs9HtB|fmT;@$=4
zz4yr>mVo?|Ib^C*o!H}|w06GKhmDtLW5>3|`;~w?0$$=`uz4%K6&;U=kqC`g$@^@=
zH0>%!Pnty${^#e6aEIEaqkb;!`I9bXJVpqp;7v+Ft@e|5rI#mUb7mBd#6n*L`8y5e
z&622!O$B@CMGSQ!7+`IWiEHW+hkM-9&dyJ(s3^HMc&Yc5^_dcvNUtJl5QNW3s&6vO
z0s!Al4hPApLt597{9c|>j^z}Y7EWtTiNeie8J^pi+M`>ew=U&|<{e-O3J@p%irfD{
z?kVNTGtlLFC|}U~P^dL#O%L?PQ6P}loH~0Jn<rbKw2QjyB~Hr~#KqgU>fB~~OspUf
zhQVS^tE<wpQ5g@l&!s|I(G>$t32l`>PytLxr&zUh*+V-hME-e=(KHmw&Xkvj*xkvA
rA0#!9aELc67R}V0cnQxU;m+5b$i>2A4)aHay0f|KeTBvxI>C);#ZG?e

diff --git a/server/config/settings/base.py b/server/config/settings/base.py
index a4fda6fd..6533d160 100644
--- a/server/config/settings/base.py
+++ b/server/config/settings/base.py
@@ -682,10 +682,12 @@ if APP_ENVIRONMENT.startswith("prod"):
     DATATRANS_PAY_URL = "https://pay.datatrans.com"
 else:
     DATATRANS_API_ENDPOINT = env(
-        "DATATRANS_API_ENDPOINT", default="https://api.sandbox.datatrans.com"
+        "DATATRANS_API_ENDPOINT",
+        default="http://localhost:8000/server/fakeapi/datatrans/api",
     )
     DATATRANS_PAY_URL = env(
-        "DATATRANS_PAY_URL", default="https://pay.sandbox.datatrans.com"
+        "DATATRANS_PAY_URL",
+        default="http://localhost:8000/server/fakeapi/datatrans/pay",
     )
 
 # default settings for python sftpserver test-server

From ab60eebd25fbdb3d1a96a0120988ee3a6078d94f Mon Sep 17 00:00:00 2001
From: Daniel Egger <daniel.egger@gmail.com>
Date: Tue, 23 Jul 2024 14:02:23 +0200
Subject: [PATCH 11/12] Use order_id as refno

---
 server/vbv_lernwelt/shop/services.py          |  6 +-
 .../shop/tests/test_checkout_api.py           | 24 +++---
 .../shop/tests/test_datatrans_service.py      | 19 ++---
 server/vbv_lernwelt/shop/views.py             | 73 ++++++++++---------
 4 files changed, 62 insertions(+), 60 deletions(-)

diff --git a/server/vbv_lernwelt/shop/services.py b/server/vbv_lernwelt/shop/services.py
index a9561d0e..0e7bd7b3 100644
--- a/server/vbv_lernwelt/shop/services.py
+++ b/server/vbv_lernwelt/shop/services.py
@@ -1,6 +1,5 @@
 import hashlib
 import hmac
-import uuid
 
 import requests
 import structlog
@@ -72,6 +71,7 @@ def is_signature_valid(
 
 def init_datatrans_transaction(
     user: User,
+    refno: str,
     amount_chf_centimes: int,
     redirect_url_success: str,
     redirect_url_error: str,
@@ -90,8 +90,8 @@ def init_datatrans_transaction(
         "amount": amount_chf_centimes,
         "currency": "CHF",
         "language": user.language,
-        "refno": str(uuid.uuid4()),
-        "refno2": refno2,
+        "refno": str(refno),
+        "refno2": str(refno2),
         "webhook": {"url": webhook_url},
         "redirect": {
             "successUrl": redirect_url_success,
diff --git a/server/vbv_lernwelt/shop/tests/test_checkout_api.py b/server/vbv_lernwelt/shop/tests/test_checkout_api.py
index efbee5da..f5f46b37 100644
--- a/server/vbv_lernwelt/shop/tests/test_checkout_api.py
+++ b/server/vbv_lernwelt/shop/tests/test_checkout_api.py
@@ -71,9 +71,8 @@ class CheckoutAPITestCase(APITestCase):
 
         # THEN
         self.assertEqual(response.status_code, status.HTTP_200_OK)
-        self.assertEqual(
-            f"https://pay.sandbox.datatrans.com/v1/start/1234567890",
-            response.json()["next_step_url"],
+        self.assertTrue(
+            response.json()["next_step_url"].endswith("v1/start/1234567890")
         )
 
         ci = CheckoutInformation.objects.first()
@@ -154,9 +153,11 @@ class CheckoutAPITestCase(APITestCase):
         )
 
         self.assertEqual(
-            0,
+            1,
             CheckoutInformation.objects.count(),
         )
+        ci = CheckoutInformation.objects.first()
+        self.assertEqual(ci.state, CheckoutState.FAILED)
 
     def test_checkout_already_paid(self):
         # GIVEN
@@ -217,9 +218,8 @@ class CheckoutAPITestCase(APITestCase):
 
         # THEN
         self.assertEqual(response.status_code, status.HTTP_200_OK)
-        self.assertEqual(
-            f"https://pay.sandbox.datatrans.com/v1/start/{transaction_id_next}",
-            response.json()["next_step_url"],
+        self.assertTrue(
+            response.json()["next_step_url"].endswith(f"v1/start/{transaction_id_next}")
         )
 
         # check that we have two checkouts
@@ -277,9 +277,8 @@ class CheckoutAPITestCase(APITestCase):
 
         # THEN
         self.assertEqual(response.status_code, status.HTTP_200_OK)
-        self.assertEqual(
-            f"https://pay.sandbox.datatrans.com/v1/start/{transaction_id}",
-            response.json()["next_step_url"],
+        self.assertTrue(
+            response.json()["next_step_url"].endswith(f"v1/start/{transaction_id}")
         )
 
     @patch("vbv_lernwelt.shop.views.init_datatrans_transaction")
@@ -310,7 +309,6 @@ class CheckoutAPITestCase(APITestCase):
 
         # THEN
         self.assertEqual(response.status_code, status.HTTP_200_OK)
-        self.assertEqual(
-            f"https://pay.sandbox.datatrans.com/v1/start/{transaction_id}",
-            response.json()["next_step_url"],
+        self.assertTrue(
+            response.json()["next_step_url"].endswith(f"v1/start/{transaction_id}")
         )
diff --git a/server/vbv_lernwelt/shop/tests/test_datatrans_service.py b/server/vbv_lernwelt/shop/tests/test_datatrans_service.py
index 6dc907fe..7f22c8ee 100644
--- a/server/vbv_lernwelt/shop/tests/test_datatrans_service.py
+++ b/server/vbv_lernwelt/shop/tests/test_datatrans_service.py
@@ -24,10 +24,8 @@ class DatatransServiceTest(TestCase):
 
     @override_settings(DATATRANS_BASIC_AUTH_KEY="BASIC_AUTH_KEY")
     @patch("vbv_lernwelt.shop.services.requests.post")
-    @patch("vbv_lernwelt.shop.services.uuid.uuid4")
-    def test_init_transaction_201(self, mock_uuid, mock_post):
+    def test_init_transaction_201(self, mock_post):
         # GIVEN
-        mock_uuid.return_value = uuid.uuid4()
         mock_post.return_value.status_code = 201
         mock_post.return_value.json.return_value = {
             "transactionId": 1234567890,
@@ -38,6 +36,7 @@ class DatatransServiceTest(TestCase):
         # WHEN
         transaction_id = init_datatrans_transaction(
             user=self.user,
+            refno="123321",
             amount_chf_centimes=324_30,
             redirect_url_success=f"{REDIRECT_URL}/success",
             redirect_url_error=f"{REDIRECT_URL}/error",
@@ -64,11 +63,12 @@ class DatatransServiceTest(TestCase):
         with self.assertRaises(InitTransactionException):
             init_datatrans_transaction(
                 user=self.user,
+                refno="123321",
                 amount_chf_centimes=324_30,
-                redirect_url_success=f"/success",
-                redirect_url_error=f"/error",
-                redirect_url_cancel=f"/cancel",
-                webhook_url=f"/webhook",
+                redirect_url_success="/success",
+                redirect_url_error="/error",
+                redirect_url_cancel="/cancel",
+                webhook_url="/webhook",
                 refno2="",
             )
 
@@ -80,7 +80,4 @@ class DatatransServiceTest(TestCase):
         url = get_payment_url(transaction_id)
 
         # THEN
-        self.assertEqual(
-            url,
-            f"https://pay.sandbox.datatrans.com/v1/start/{transaction_id}",
-        )
+        self.assertTrue(url.endswith(f"v1/start/{transaction_id}"))
diff --git a/server/vbv_lernwelt/shop/views.py b/server/vbv_lernwelt/shop/views.py
index 1ce904a3..a245520a 100644
--- a/server/vbv_lernwelt/shop/views.py
+++ b/server/vbv_lernwelt/shop/views.py
@@ -93,7 +93,7 @@ def checkout_vv(request):
     sku = request.data["product"]
     base_redirect_url = request.data["redirect_url"]
 
-    log.info(f"Checkout requested: sku", user_id=request.user.id, sku=sku)
+    log.info("Checkout requested: sku", user_id=request.user.id, sku=sku)
 
     try:
         product = Product.objects.get(sku=sku)
@@ -124,6 +124,38 @@ def checkout_vv(request):
         disable_save="fakeapi" in settings.DATATRANS_API_ENDPOINT
     )
 
+    address_data = request.data["address"]
+    country_code = address_data.pop("country_code")
+    address_data["country_id"] = country_code
+
+    organisation_country_code = "CH"
+    if "organisation_country_code" in address_data:
+        organisation_country_code = address_data.pop("organisation_country_code")
+    address_data["organisation_country_id"] = organisation_country_code
+
+    if "birth_date" in address_data and address_data["birth_date"]:
+        address_data["birth_date"] = date.fromisoformat(address_data["birth_date"])
+
+    checkout_info = CheckoutInformation.objects.create(
+        user=request.user,
+        state=CheckoutState.ONGOING,
+        # product
+        product_sku=sku,
+        product_price=product.price,
+        product_name=product.name,
+        product_description=product.description,
+        email=email,
+        ip_address=ip_address,
+        cembra_byjuno_invoice=with_cembra_byjuno_invoice,
+        device_fingerprint_session_key=request.data.get(
+            "device_fingerprint_session_key", ""
+        ),
+        # address
+        **request.data["address"],
+    )
+
+    checkout_info.set_increment_abacus_order_id()
+
     refno2 = f"{request.user.abacus_debitor_number}_{VV_PRODUCT_NUMBER}"
 
     try:
@@ -138,10 +170,10 @@ def checkout_vv(request):
                 "street": f'{request.data["address"]["street"]} {request.data["address"]["street_number"]}',
                 "city": request.data["address"]["city"],
                 "zipCode": request.data["address"]["postal_code"],
-                "country": request.data["address"]["country_code"],
+                "country": request.data["address"]["country_id"],
                 "phone": request.data["address"]["phone_number"],
                 "email": email,
-                "birthDate": request.data["address"]["birth_date"],
+                "birthDate": str(request.data["address"]["birth_date"]),
                 "language": request.user.language,
                 "ipAddress": ip_address,
                 "type": "P",
@@ -154,6 +186,7 @@ def checkout_vv(request):
             }
         transaction_id = init_datatrans_transaction(
             user=request.user,
+            refno=str(checkout_info.abacus_order_id),
             amount_chf_centimes=product.price,
             redirect_url_success=checkout_success_url(
                 base_url=base_redirect_url, product_sku=sku
@@ -170,6 +203,8 @@ def checkout_vv(request):
             with_cembra_byjuno_invoice=with_cembra_byjuno_invoice,
         )
     except InitTransactionException as e:
+        checkout_info.state = CheckoutState.FAILED.value
+        checkout_info.save()
         if not settings.DEBUG:
             log.error("Transaction initiation failed", exc_info=True, error=str(e))
             capture_exception(e)
@@ -180,36 +215,8 @@ def checkout_vv(request):
             ),
         )
 
-    address_data = request.data["address"]
-    country_code = address_data.pop("country_code")
-    address_data["country_id"] = country_code
-
-    organisation_country_code = "CH"
-    if "organisation_country_code" in address_data:
-        organisation_country_code = address_data.pop("organisation_country_code")
-    address_data["organisation_country_id"] = organisation_country_code
-
-    if "birth_date" in address_data and address_data["birth_date"]:
-        address_data["birth_date"] = date.fromisoformat(address_data["birth_date"])
-
-    checkout_info = CheckoutInformation.objects.create(
-        user=request.user,
-        state=CheckoutState.ONGOING,
-        transaction_id=transaction_id,
-        # product
-        product_sku=sku,
-        product_price=product.price,
-        product_name=product.name,
-        product_description=product.description,
-        email=email,
-        ip_address=ip_address,
-        cembra_byjuno_invoice=with_cembra_byjuno_invoice,
-        device_fingerprint_session_key=request.data.get(
-            "device_fingerprint_session_key", ""
-        ),
-        # address
-        **request.data["address"],
-    )
+    checkout_info.transaction_id = transaction_id
+    checkout_info.save()
 
     return next_step_response(url=get_payment_url(transaction_id))
 

From 08b65f00d743fd8cc2a785280df51abc47c4339a Mon Sep 17 00:00:00 2001
From: Daniel Egger <daniel.egger@gmail.com>
Date: Tue, 23 Jul 2024 14:39:49 +0200
Subject: [PATCH 12/12] Enable cembra/byjuno payment for PROD env

---
 client/src/components/onboarding/PersonalAddress.vue       | 7 -------
 .../learningContentPage/assignment/AssignmentView.vue      | 4 ----
 server/config/settings/base.py                             | 1 -
 3 files changed, 12 deletions(-)

diff --git a/client/src/components/onboarding/PersonalAddress.vue b/client/src/components/onboarding/PersonalAddress.vue
index e40dd405..0430d18a 100644
--- a/client/src/components/onboarding/PersonalAddress.vue
+++ b/client/src/components/onboarding/PersonalAddress.vue
@@ -32,13 +32,6 @@ const paymentMethods = [
   { value: "cembra_byjuno", label: t("a.Rechnung") },
 ];
 
-// TODO: remove after cembra is ready for production
-const appEnv = import.meta.env.VITE_APP_ENVIRONMENT || "local";
-if (appEnv.startsWith("prod")) {
-  paymentMethods.splice(1, 1);
-}
-// END TODO
-
 const address = computed({
   get() {
     return props.modelValue;
diff --git a/client/src/pages/learningPath/learningContentPage/assignment/AssignmentView.vue b/client/src/pages/learningPath/learningContentPage/assignment/AssignmentView.vue
index d6d79516..48ae62a3 100644
--- a/client/src/pages/learningPath/learningContentPage/assignment/AssignmentView.vue
+++ b/client/src/pages/learningPath/learningContentPage/assignment/AssignmentView.vue
@@ -51,10 +51,6 @@ const submissionDeadline = computed(() => {
     ?.submission_deadline;
 });
 
-// FIXME daniel: `useRouteQuery` from usevue is currently the reason that we have to
-// fix the version of @vueuse/router and @vueuse/core to 10.1.0
-// it fails with version 10.2.0. I have a reminder to check out the situation
-// at the end of July 2023
 // 0 = introduction, 1 - n = tasks, n+1 = submission
 const stepIndex = useRouteQuery("step", "0", { transform: Number, mode: "push" });
 
diff --git a/server/config/settings/base.py b/server/config/settings/base.py
index 6533d160..031cb063 100644
--- a/server/config/settings/base.py
+++ b/server/config/settings/base.py
@@ -332,7 +332,6 @@ X_FRAME_OPTIONS = "DENY"
 # EMAIL
 # ------------------------------------------------------------------------------
 # https://docs.djangoproject.com/en/dev/ref/settings/#email-backend
-# FIXME how to send emails?
 EMAIL_BACKEND = env(
     "DJANGO_EMAIL_BACKEND", default="django.core.mail.backends.console.EmailBackend"
 )