Update types for backup endpoints

packages/builder/src/stores/portal/backups.js

@@ -19,7 +19,7 @@ export function createBackupsStore() {
     startDate,
     endDate,
   }) {
-    return API.searchBackups({ appId, trigger, type, page, startDate, endDate })
+    return API.searchBackups(appId, { trigger, type, page, startDate, endDate })
   }
 
   async function restoreBackup({ appId, backupId, name }) {

packages/frontend-core/src/api/backups.ts

@@ -1,60 +1,56 @@
-import { PutResponse } from "@budibase/types"
+import {
+  CreateAppBackupRequest,
+  CreateAppBackupResponse,
+  ImportAppBackupResponse,
+  SearchAppBackupsRequest,
+  UpdateAppBackupRequest,
+} from "@budibase/types"
 import { BaseAPIClient } from "./types"
 
 export interface BackupEndpoints {
   createManualBackup: (
-    appId: string
-  ) => Promise<{ backupId: string; message: string }>
-  deleteBackup: (
     appId: string,
-    backupId: string
+    name?: string
-  ) => Promise<{ message: string }>
+  ) => Promise<CreateAppBackupResponse>
-  updateBackup: (
-    appId: string,
-    backupId: string,
-    name: string
-  ) => Promise<PutResponse>
   restoreBackup: (
     appId: string,
     backupId: string,
     name: string
-  ) => Promise<{ restoreId: string; message: string }>
+  ) => Promise<ImportAppBackupResponse>
 
   // Missing request or response types
-  searchBackups: (opts: any) => Promise<any>
+  searchBackups: (appId: string, opts: SearchAppBackupsRequest) => Promise<any>
+  updateBackup: (appId: string, backupId: string, name: string) => Promise<any>
+  deleteBackup: (
+    appId: string,
+    backupId: string
+  ) => Promise<{ message: string }>
 }
 
 export const buildBackupEndpoints = (API: BaseAPIClient): BackupEndpoints => ({
-  searchBackups: async ({ appId, trigger, type, page, startDate, endDate }) => {
+  createManualBackup: async (appId, name) => {
-    const opts: any = {}
+    return await API.post<CreateAppBackupRequest, CreateAppBackupResponse>({
-    if (page) {
+      url: `/api/apps/${appId}/backups`,
-      opts.page = page
+      body: name
-    }
+        ? {
-    if (trigger && type) {
+            name,
-      opts.trigger = trigger.toLowerCase()
-      opts.type = type.toLowerCase()
-    }
-    if (startDate && endDate) {
-      opts.startDate = startDate
-      opts.endDate = endDate
           }
+        : undefined,
+    })
+  },
+  searchBackups: async (appId, opts) => {
     return await API.post({
       url: `/api/apps/${appId}/backups/search`,
       body: opts,
     })
   },
-  createManualBackup: async appId => {
-    return await API.post({
-      url: `/api/apps/${appId}/backups`,
-    })
-  },
   deleteBackup: async (appId, backupId) => {
     return await API.delete({
       url: `/api/apps/${appId}/backups/${backupId}`,
     })
   },
   updateBackup: async (appId, backupId, name) => {
-    return await API.patch({
+    return await API.patch<UpdateAppBackupRequest, any>({
       url: `/api/apps/${appId}/backups/${backupId}`,
       body: { name },
     })