trigger recursive sync when connecting a new storage location

Author: TimCsaky

frontend/src/components/bucket/BucketConfigForm.vue

@@ -78,10 +78,13 @@ const onSubmit = async (values: any) => {
 
     const bucketChanges = differential(formBucket, initialValues);
 
-    props.bucket
+    const bucketModel = props.bucket
       ? await bucketStore.updateBucket(props.bucket?.bucketId, bucketChanges)
       : await bucketStore.createBucket(formBucket);
 
+    // if successfully added a new configuration, do a recursive sync of this bucket
+    if(!props.bucket) await bucketStore.syncBucket(bucketModel.bucketId, true);
+    // refresh bucket list
     await bucketStore.fetchBuckets({ userId: getUserId.value, objectPerms: true });
 
     // trim trailing "//", if present

frontend/src/components/bucket/BucketTable.vue

@@ -240,8 +240,7 @@ watch(getBuckets, () => {
     }
   }
 
-  // Expand all nodes and set tree state
-  bucketTreeNodeMap.forEach((_v, k) => (expandedKeys.value[k] = true));
+  //set tree state
   treeData.value = tree;
 });
 </script>

frontend/src/components/common/SyncButton.vue

@@ -41,7 +41,7 @@ const onSubmit = () => {
   if (props.objectId) {
     objectStore.syncObject(props.objectId);
   } else if (props.bucketId) {
-    bucketStore.syncBucket(props.bucketId);
+    bucketStore.syncBucket(props.bucketId, false);
   } else {
     toast.error('', 'Unable to synchronize');
   }

frontend/src/components/object/ObjectVersion.vue

@@ -47,12 +47,14 @@ const isDeleted: Ref<boolean> = computed(() => getIsDeleted.value(props.objectId
 const tableData: Ref<Array<VersionDataSource>> = computed(() => {
   return versions.value
     .filter(v => !v.deleteMarker)
+    .sort((a, b) => {
+      return new Date(a.lastModifiedDate) < new Date(b.lastModifiedDate) ? 1 : -1;
+    })
     .map((v: Version, index, arr) => ({
       ...v,
       createdByName: getUser.value(v.createdBy)?.fullName,
       versionNumber: arr.length - index,
       isDeleted:  isDeleted.value,
-
     }));
 });
 // Highlight row for currently selected version

frontend/src/services/bucketService.ts

@@ -64,7 +64,11 @@ export default {
    * @param {string} bucketId Bucket ID for the bucket to synchronize
    * @returns {Promise} An axios response
    */
-  syncBucket(bucketId: string) {
-    return comsAxios().get(`${BUCKET_PATH}/${bucketId}/sync`);
+  syncBucket(bucketId: string, recursive: boolean ) {
+    return comsAxios().get(`${BUCKET_PATH}/${bucketId}/sync`, {
+      params: {
+        recursive: recursive,
+      }
+    });
   }
 };

frontend/src/store/bucketStore.ts

@@ -112,11 +112,11 @@ export const useBucketStore = defineStore('bucket', () => {
     }
   }
 
-  async function syncBucket(bucketId: string) {
+  async function syncBucket(bucketId: string, recursive: boolean) {
     try {
       appStore.beginIndeterminateLoading();
 
-      await bucketService.syncBucket(bucketId);
+      await bucketService.syncBucket(bucketId, recursive );
       toast.success('', 'Sync is in queue and will begin soon');
     } catch (error: any) {
       toast.error('Unable to sync', error);

frontend/src/types/Version.ts

@@ -8,4 +8,5 @@ export type Version = {
   mimeType: string;
   objectId: string;
   s3VersionId: string;
+  lastModifiedDate: string;
 } & IAudit;

frontend/tests/unit/store/versionStore.spec.ts

@@ -31,7 +31,8 @@ const version: Version = {
   objectId: '000',
   s3VersionId: 's3123',
   isLatest: true,
-  createdAt: '2023-05-01T22:18:12.553Z'
+  createdAt: '2023-05-01T22:18:12.553Z',
+  lastModifiedDate: '2023-05-01T22:18:12.553Z',
 };
 
 const versionOld: Version = {
@@ -41,7 +42,8 @@ const versionOld: Version = {
   objectId: '000',
   s3VersionId: 's2000',
   isLatest: false,
-  createdAt: '2022-05-01T18:25:42.462Z'
+  createdAt: '2022-05-01T18:25:42.462Z',
+  lastModifiedDate: '2023-05-01T22:18:12.553Z',
 };
 
 const mockToast = vi.fn();