From 2f9b82ca4475964ba4679c4e4f46552c4d4b2b5d Mon Sep 17 00:00:00 2001 From: soup Date: Sat, 4 May 2024 11:21:00 +0200 Subject: [PATCH] refactor(apiclient): simplify query in release delete method (#1538) * refactor(apiclient): simplify query in delete method * refactor(apiclient): simplify query in delete method --- web/src/api/APIClient.ts | 25 +++++-------------------- web/src/types/Release.d.ts | 6 ++++++ 2 files changed, 11 insertions(+), 20 deletions(-) diff --git a/web/src/api/APIClient.ts b/web/src/api/APIClient.ts index b87784e..fd4e774 100644 --- a/web/src/api/APIClient.ts +++ b/web/src/api/APIClient.ts @@ -15,14 +15,6 @@ interface HttpConfig { queryString?: Record; } -interface DeleteParams { - olderThan?: number; - indexers?: string[]; - releaseStatuses?: string[]; -} - -type QueryStringParams = Record; - // See https://stackoverflow.com/a/62969380 function encodeRFC3986URIComponent(str: string): string { return encodeURIComponent(str).replace( @@ -347,19 +339,12 @@ export const APIClient = { indexerOptions: () => appClient.Get("api/release/indexers"), stats: () => appClient.Get("api/release/stats"), delete: (params: DeleteParams) => { - const queryString: QueryStringParams = {}; - if (params.olderThan !== undefined) { - queryString.olderThan = params.olderThan.toString(); - } - if (params.indexers && params.indexers.length > 0) { - queryString.indexer = params.indexers; - } - if (params.releaseStatuses && params.releaseStatuses.length > 0) { - queryString.releaseStatus = params.releaseStatuses; - } - return appClient.Delete("api/release", { - queryString + queryString: { + olderThan: params.olderThan, + indexer: params.indexers, + releaseStatus: params.releaseStatuses, + } }); }, replayAction: (releaseId: number, actionId: number) => appClient.Post( diff --git a/web/src/types/Release.d.ts b/web/src/types/Release.d.ts index b32ce6d..369551a 100644 --- a/web/src/types/Release.d.ts +++ b/web/src/types/Release.d.ts @@ -68,3 +68,9 @@ interface ReleaseFilter { id: string; value: string; } + +interface DeleteParams { + olderThan?: number; + indexers?: string[]; + releaseStatuses?: string[]; +} \ No newline at end of file