From e15654b2a5c00ce10f8e8dfd846bd36293dbed5d Mon Sep 17 00:00:00 2001
From: Mel <97147377+MelissaAutumn@users.noreply.github.com>
Date: Mon, 15 Jan 2024 09:10:16 -0800
Subject: [PATCH] Features/228 signed url store (#229)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* ➕ move navbar avatar to separate component

(cherry picked from commit 2d5dc901b9f5c1a515307eafd83ff7dda10dd6ad)

* ➕ add signed url to user store

(cherry picked from commit 5adee893e848ff9993b31d1074bd2471a315e067)

---------

Co-authored-by: Andreas Müller <mail@devmount.de>
---
 frontend/src/components/NavBar.vue           | 38 ++++----------------
 frontend/src/components/ScheduleCreation.vue | 25 +++----------
 frontend/src/components/SettingsAccount.vue  | 29 ++++-----------
 frontend/src/elements/Avatar.vue             | 19 ++++++++++
 frontend/src/stores/user-store.js            | 27 +++++++++++++-
 5 files changed, 61 insertions(+), 77 deletions(-)
 create mode 100644 frontend/src/elements/Avatar.vue

diff --git a/frontend/src/components/NavBar.vue b/frontend/src/components/NavBar.vue
index cc7d0c32a..0d71129ef 100644
--- a/frontend/src/components/NavBar.vue
+++ b/frontend/src/components/NavBar.vue
@@ -37,17 +37,7 @@
       </div>
       <drop-down class="self-center">
         <template #trigger>
-          <div
-            class="w-12 h-12 mr-4 self-center flex-center rounded-full text-lg bg-white font-normal text-white"
-            :class="{'bg-teal-500': user.data.avatarUrl === null}"
-          >
-            <span v-if="user.data.avatarUrl === null">
-              {{ initials(user.data.name) }}
-            </span>
-            <span v-else>
-              <img class="rounded-full w-[48px] h-[48px]" :alt="initials(user.data.name)" :src="user.data.avatarUrl"/>
-            </span>
-          </div>
+          <avatar />
         </template>
         <template #default>
           <div class="flex flex-col gap-2 rounded-md w-48 p-4 bg-white dark:bg-gray-700 shadow-md">
@@ -55,9 +45,9 @@
               {{ t('label.userProfile') }}
             </router-link>
             <text-button
-              v-show="signedUserUrl"
+              v-show="user.data.signedUrl"
               :label="t('label.shareMyLink')"
-              :copy="signedUserUrl"
+              :copy="user.data.signedUrl"
               class="border-none flex-row-reverse justify-between !text-inherit !text-base !font-normal hover:bg-inherit hover:shadow-none"
             />
             <hr class="border-teal-500" />
@@ -72,13 +62,13 @@
 </template>
 
 <script setup>
-import { ref, inject, onMounted } from 'vue';
+import { inject } from 'vue';
 import { useI18n } from 'vue-i18n';
 import { useRoute, useRouter } from 'vue-router';
-import { initials } from '@/utils';
 import { useUserStore } from '@/stores/user-store';
-import NavBarItem from "@/elements/NavBarItem";
+import Avatar from "@/elements/Avatar";
 import DropDown from "@/elements/DropDown";
+import NavBarItem from "@/elements/NavBarItem";
 import TextButton from "@/elements/TextButton";
 
 // component constants
@@ -93,25 +83,9 @@ defineProps({
   navItems: Array, // list of route names that are also lang keys (format: label.<key>), used as nav items
 });
 
-const signedUserUrl = ref('');
-
 // do log out
 const logout = async () => {
   await user.logout(call);
   await router.push('/');
 };
-
-const getSignedUserUrl = async () => {
-  // Retrieve the user short url
-  const { data, error } = await call('me/signature').get().json();
-  if (error.value) {
-    return;
-  }
-
-  signedUserUrl.value = data.value.url;
-};
-
-onMounted(async () => {
-  await getSignedUserUrl();
-});
 </script>
diff --git a/frontend/src/components/ScheduleCreation.vue b/frontend/src/components/ScheduleCreation.vue
index e517831a2..62d5b8d97 100644
--- a/frontend/src/components/ScheduleCreation.vue
+++ b/frontend/src/components/ScheduleCreation.vue
@@ -267,9 +267,9 @@
     <!-- action buttons -->
     <div class="flex justify-center gap-4 mt-auto">
       <primary-button
-        v-if="publicLink && existing"
+        v-if="user.data.signedUrl && existing"
         :label="t('label.shareMyLink')"
-        :copy="publicLink"
+        :copy="user.data.signedUrl"
       />
     </div>
     <div class="flex gap-4 mt-auto">
@@ -301,7 +301,7 @@
     :open="savedConfirmation.show"
     :is-schedule="true"
     :title="savedConfirmation.title"
-    :public-link="publicLink"
+    :public-link="user.data.signedUrl"
     @close="closeCreatedModal"
   />
 </template>
@@ -475,15 +475,6 @@ const resetSchedule = () => {
   state.value = scheduleCreationState.details;
 };
 
-// hold public availability link of user
-const publicLink = ref('');
-onMounted(async () => {
-  const { data, error } = await call('me/signature').get().json();
-  if (!error.value) {
-    publicLink.value = data.value.url;
-  }
-});
-
 // handle actual schedule creation/update
 const savingInProgress = ref(false);
 const saveSchedule = async (withConfirmation = true) => {
@@ -519,17 +510,9 @@ const saveSchedule = async (withConfirmation = true) => {
     return;
   }
 
-  if (withConfirmation) {
-    // Retrieve the user short url
-    const { data: sig_data, error: sig_error } = await call('me/signature').get().json();
-    if (sig_error.value) {
-      savingInProgress.value = false;
-      return;
-    }
-  
+  if (withConfirmation) {  
     // show confirmation
     savedConfirmation.title = data.value.name;
-    publicLink.value = sig_data.value.url;
     savedConfirmation.show = true;
   }
 
diff --git a/frontend/src/components/SettingsAccount.vue b/frontend/src/components/SettingsAccount.vue
index 0d59918c4..fe6fe9dfb 100644
--- a/frontend/src/components/SettingsAccount.vue
+++ b/frontend/src/components/SettingsAccount.vue
@@ -30,16 +30,16 @@
         <div class="w-full flex justify-between items-center gap-4">
           <div class="relative w-full">
             <input
-              v-model="signedUserUrl"
+              :value="user.data.signedUrl"
               type="text"
               class="w-full rounded-md mr-2 pr-7"
               readonly
             />
-            <a :href="signedUserUrl" target="_blank" class="text-gray-500 absolute right-1.5 top-1/2 -translate-y-1/2">
+            <a :href="user.data.signedUrl" target="_blank" class="text-gray-500 absolute right-1.5 top-1/2 -translate-y-1/2">
               <icon-external-link class="w-5 h-5" />
             </a>
           </div>
-          <text-button :label="t('label.copyLink')" :copy="signedUserUrl" />
+          <text-button :label="t('label.copyLink')" :copy="user.data.signedUrl" />
         </div>
       </label>
       <div class="self-end flex gap-4 mt-6">
@@ -177,9 +177,6 @@ const deleteAccountSecondModalOpen = ref(false);
 const refreshLinkModalOpen = ref(false);
 const updateUsernameModalOpen = ref(false);
 
-// calculate signed link
-const signedUserUrl = ref('');
-
 const closeModals = () => {
   downloadAccountModalOpen.value = false;
   deleteAccountFirstModalOpen.value = false;
@@ -188,18 +185,8 @@ const closeModals = () => {
   updateUsernameModalOpen.value = false;
 };
 
-const getSignedUserUrl = async () => {
-  // Retrieve the user short url
-  const { data, error } = await call('me/signature').get().json();
-  if (error.value) {
-    return;
-  }
-
-  signedUserUrl.value = data.value.url;
-};
-
 const refreshData = async () => Promise.all([
-  getSignedUserUrl(),
+  user.updateSignedUrl(call),
   externalConnectionsStore.fetch(call),
 ]);
 
@@ -214,6 +201,7 @@ const updateUser = async () => {
   if (!error.value) {
     // update user in store
     user.$patch({ data: { ...user.data, ...inputData }});
+    await user.updateSignedUrl(call);
     errorUsername.value = false;
     // TODO show some confirmation
     await refreshData();
@@ -270,12 +258,7 @@ const refreshLink = async () => {
 };
 
 const refreshLinkConfirm = async () => {
-  const { data, error } = await call('me/signature').post().json();
-
-  if (error.value) {
-    console.log('Error!', data.value);
-  }
-
+  await user.changeSignedUrl(call);
   await refreshData();
   closeModals();
 };
diff --git a/frontend/src/elements/Avatar.vue b/frontend/src/elements/Avatar.vue
new file mode 100644
index 000000000..f114e2668
--- /dev/null
+++ b/frontend/src/elements/Avatar.vue
@@ -0,0 +1,19 @@
+<template>
+<div
+  class="w-12 h-12 mr-4 self-center flex-center rounded-full text-lg bg-white font-normal text-white"
+  :class="{'!bg-teal-500': user.data.avatarUrl === null}"
+>
+  <span v-if="user.data.avatarUrl === null">
+    {{ initials(user.data.name) }}
+  </span>
+  <span v-else>
+    <img class="rounded-full w-[48px] h-[48px]" :alt="initials(user.data.name)" :src="user.data.avatarUrl"/>
+  </span>
+</div>
+</template>
+
+<script setup>
+import { initials } from '@/utils';
+import { useUserStore } from '@/stores/user-store';
+const user = useUserStore();
+</script>
diff --git a/frontend/src/stores/user-store.js b/frontend/src/stores/user-store.js
index c9387a59b..a62a726e3 100644
--- a/frontend/src/stores/user-store.js
+++ b/frontend/src/stores/user-store.js
@@ -7,6 +7,7 @@ const initialUserObject = {
   name: null,
   timezone: null,
   username: null,
+  signedUrl: null,
   avatarUrl: null,
   accessToken: null,
 };
@@ -42,8 +43,32 @@ export const useUserStore = defineStore('user', {
         }
       });
 
+      return await this.signedUrl(fetch);
+    },
+    // retrieve the current signed url and update store
+    async updateSignedUrl(fetch) {
+      const { error, data } = await fetch('me/signature').get().json();
+
+      if (error.value || !data.value.url) {
+        console.error(error.value, data.value);
+        return false;
+      }
+
+      this.data.signedUrl = data.value.url;
+
       return true;
     },
+    // invalidate the current signed url and replace it with a new one
+    async changeSignedUrl(fetch) {
+      const { error, data } = await fetch('me/signature').post().json();
+
+      if (error.value) {
+        console.error(error.value, data.value);
+        return false;
+      }
+
+      return this.updateSignedUrl(fetch);
+    },
     async login(fetch, username, password) {
       this.reset();
 
@@ -67,7 +92,7 @@ export const useUserStore = defineStore('user', {
       return await this.profile(fetch);
     },
     async logout(fetch) {
-      const { error, data } = await fetch('logout').get().json();
+      const { error } = await fetch('logout').get().json();
 
       if (error.value) {
         console.warn("Error logging out: ", error.value);