fix(server): unable delete cronjob in k8s (#873)

labring · Mar 8, 2023 · 3b015d3 · 3b015d3
1 parent b096734
commit 3b015d3
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/server/src/trigger/trigger.controller.ts b/server/src/trigger/trigger.controller.ts
@@ -85,8 +85,8 @@ export class TriggerController {
   @ApiResponse({ type: ResponseUtil })
   @UseGuards(JwtAuthGuard, ApplicationAuthGuard)
   @Delete(':id')
-  async remove(@Param('appid') appid: string, @Param('id') id: string) {
-    const res = await this.triggerService.remove(appid, id)
+  async remove(@Param('id') id: string) {
+    const res = await this.triggerService.remove(id)
     return ResponseUtil.ok(res)
   }
 }
diff --git a/server/src/trigger/trigger.service.ts b/server/src/trigger/trigger.service.ts
@@ -50,9 +50,12 @@ export class TriggerService {
     return res
   }
 
-  async remove(appid: string, id: string) {
-    const res = await this.prisma.cronTrigger.deleteMany({
-      where: { appid, id },
+  async remove(id: string) {
+    const res = await this.prisma.cronTrigger.update({
+      where: { id },
+      data: {
+        state: TriggerState.Deleted,
+      },
     })
     return res
   }