Merge pull request #90 from UpstreamDataInc/mm/fix-software-deletion

b-rowan · web-flow · commit 118d176c8555 · 2024-08-27T09:26:48.000-06:00
Fix software deletion from UI
diff --git a/goosebit/api/v1/software/requests.py b/goosebit/api/v1/software/requests.py
@@ -1,5 +1,5 @@
-from pydantic import BaseModel
+from pydantic import RootModel
 
 
-class SoftwareDeleteRequest(BaseModel):
-    files: list[int]
+class SoftwareDeleteRequest(RootModel[list[int]]):
+    pass
diff --git a/goosebit/api/v1/software/routes.py b/goosebit/api/v1/software/routes.py
@@ -26,7 +26,7 @@ async def software_get(_: Request) -> SoftwareResponse:
 )
 async def software_delete(_: Request, config: SoftwareDeleteRequest) -> StatusResponse:
     success = False
-    for f_id in config.files:
+    for f_id in config.root:
         software = await Software.get_or_none(id=f_id)
 
         if software is None:

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ async def software_get(_: Request) -> SoftwareResponse:`
`26`	`26`	`)`
`27`	`27`	`async def software_delete(_: Request, config: SoftwareDeleteRequest) -> StatusResponse:`
`28`	`28`	`success = False`
`29`		`- for f_id in config.files:`
	`29`	`+ for f_id in config.root:`
`30`	`30`	`software = await Software.get_or_none(id=f_id)`
`31`	`31`
`32`	`32`	`if software is None:`