From 9c1f214c8de768ec729207615ad1beb46fa63097 Mon Sep 17 00:00:00 2001
From: braks <78412429+bcakmakoglu@users.noreply.github.com>
Date: Thu, 21 Jul 2022 19:07:00 +0200
Subject: [PATCH] refactor(gui-v2): cleanup audit tab issues and styles

# What's changed?

* load project before loading audits
* properly align pagination
* move reload button to pagination
* wrap elements
* prevent warnings of ant design by using functions as children
* add icon to reload
---
 .../dashboard/settings/AuditTab.vue           | 88 +++++++++++--------
 packages/nc-gui-v2/utils/dateTimeUtils.ts     |  2 +-
 2 files changed, 53 insertions(+), 37 deletions(-)

diff --git a/packages/nc-gui-v2/components/dashboard/settings/AuditTab.vue b/packages/nc-gui-v2/components/dashboard/settings/AuditTab.vue
index 4ae6cd9365..e43f07860f 100644
--- a/packages/nc-gui-v2/components/dashboard/settings/AuditTab.vue
+++ b/packages/nc-gui-v2/components/dashboard/settings/AuditTab.vue
@@ -1,38 +1,52 @@
 <script setup lang="ts">
 import { Tooltip as ATooltip } from 'ant-design-vue'
-import { h, ref } from 'vue'
 import type { AuditType } from 'nocodb-sdk'
-import { calculateDiff } from '~/utils/dateTimeUtils'
+import { timeAgo } from '~/utils/dateTimeUtils'
+import { h, ref, useNuxtApp, useProject } from '#imports'
+import MdiReload from '~icons/mdi/reload'
+
+interface Props {
+  projectId: string
+}
+
+const { projectId } = defineProps<Props>()
 
 const { $api } = useNuxtApp()
-const { project } = useProject()
+const { project, loadProject } = useProject()
+
+let isLoading = $ref(false)
+
+let audits = $ref<null | Array<AuditType>>(null)
 
-const isLoading = ref(true)
-const audits = ref<null | Array<AuditType>>(null)
-const totalRows = ref(0)
-const page = ref(1)
-const limit = ref(25)
+let totalRows = $ref(0)
 
-const loadAudits = async () => {
+const currentPage = ref(1)
+const currentLimit = ref(25)
+
+async function loadAudits(page: number, limit: number) {
   try {
-    isLoading.value = true
-    const { list, pageInfo } = await $api.project.auditList(project.value.id ?? '', {
-      offset: (limit.value * (page.value - 1)).toString(),
-      limit: limit.value.toString(),
+    if (!project.value?.id) return
+
+    isLoading = true
+
+    const { list, pageInfo } = await $api.project.auditList(project.value?.id, {
+      offset: (limit * (page - 1)).toString(),
+      limit: limit.toString(),
     })
 
-    audits.value = list
-    totalRows.value = pageInfo.totalRows ?? 0
+    audits = list
+    totalRows = pageInfo.totalRows ?? 0
   } catch (e) {
     console.error(e)
   } finally {
-    isLoading.value = false
+    isLoading = false
   }
 }
 
 onMounted(async () => {
-  if (audits.value === null) {
-    loadAudits()
+  if (audits === null) {
+    await loadProject(projectId)
+    await loadAudits(page.value, limit.value)
   }
 })
 
@@ -56,7 +70,7 @@ const columns = [
     title: 'User',
     dataIndex: 'user',
     key: 'user',
-    customRender: (value: { text: string }) => h('div', value.text || 'Shared base'),
+    customRender: (value: { text: string }) => h('div', () => value.text || 'Shared base'),
   },
   {
     title: 'Created',
@@ -64,28 +78,30 @@ const columns = [
     key: 'created_at',
     sort: 'desc',
     customRender: (value: { text: string }) =>
-      h(ATooltip, { placement: 'bottom', title: h('span', {}, value.text) }, calculateDiff(value.text)),
+      h(ATooltip, { placement: 'bottom', title: h('span', {}, value.text) }, () => timeAgo(value.text)),
   },
 ]
 </script>
 
 <template>
-  <a-button class="mb-2" :loading="isLoading" @click="loadAudits"> Reload </a-button>
+  <div class="flex flex-col items-center justify-center gap-4">
+    <a-table class="centre" :data-source="audits ?? []" :columns="columns" :pagination="false" :loading="isLoading" />
 
-  <div v-if="isLoading" class="flex flex-row justify-center w-full p-32">
-    <a-spin size="large" />
+    <div class="flex flex-wrap items-center justify-center gap-4">
+      <a-button class="self-start" @click="loadAudits">
+        <div class="flex items-center gap-2">
+          <MdiReload :class="{ 'animate-infinite animate-spin !text-success': isLoading }" />
+          Reload
+        </div>
+      </a-button>
+
+      <a-pagination
+        v-model:current="currentPage"
+        :page-size="currentLimit"
+        :total="totalRows"
+        show-less-items
+        @change="loadAudits"
+      />
+    </div>
   </div>
-  <template v-else>
-    <a-table class="centre" :data-source="audits ?? []" :columns="columns" :pagination="false" />
-    <a-pagination
-      v-model:current="page"
-      :page-size="limit"
-      class="pt-4"
-      :total="totalRows"
-      show-less-items
-      @change="loadAudits"
-    />
-  </template>
 </template>
-
-<style scoped></style>
diff --git a/packages/nc-gui-v2/utils/dateTimeUtils.ts b/packages/nc-gui-v2/utils/dateTimeUtils.ts
index 146d70a9c8..e545ee290c 100644
--- a/packages/nc-gui-v2/utils/dateTimeUtils.ts
+++ b/packages/nc-gui-v2/utils/dateTimeUtils.ts
@@ -5,7 +5,7 @@ import utc from 'dayjs/plugin/utc'
 dayjs.extend(utc)
 dayjs.extend(relativeTime)
 
-export const calculateDiff = (date: any) => {
+export const timeAgo = (date: any) => {
   return dayjs.utc(date).fromNow()
 }