Use tryExecute for delete API call

madsrasmussen · nielslyngsoe · commit 194fee7c916a · 2025-10-22T14:31:41.000+02:00
Replaces direct await of #delete with tryExecute to improve error handling in the delete method of UmbManagementApiDetailDataRequestManager.
diff --git a/src/Umbraco.Web.UI.Client/src/packages/management-api/detail/detail-data.request-manager.ts b/src/Umbraco.Web.UI.Client/src/packages/management-api/detail/detail-data.request-manager.ts
@@ -125,7 +125,7 @@ export class UmbManagementApiDetailDataRequestManager<
 	}
 
 	async delete(id: string): Promise<UmbApiWithErrorResponse> {
-		const { error } = await this.#delete(id);
+		const { error } = await tryExecute(this, this.#delete(id));
 
 		// Only update the cache when we are connected to the server events
 		if (this.#isConnectedToServerEvents && !error) {

Original file line number	Diff line number	Diff line change
`@@ -125,7 +125,7 @@ export class UmbManagementApiDetailDataRequestManager<`
`125`	`125`	`}`
`126`	`126`
`127`	`127`	`async delete(id: string): Promise<UmbApiWithErrorResponse> {`
`128`		`- const { error } = await this.#delete(id);`
	`128`	`+ const { error } = await tryExecute(this, this.#delete(id));`
`129`	`129`
`130`	`130`	`// Only update the cache when we are connected to the server events`
`131`	`131`	`if (this.#isConnectedToServerEvents && !error) {`