docs update

elastic · Aug 8, 2024 · 364d674 · 364d674
1 parent e40c095
commit 364d674
Show file tree

Hide file tree

Showing 3 changed files with 38 additions and 12 deletions.
diff --git a/docs/api/synthetics/monitors/delete-monitor-api.asciidoc b/docs/api/synthetics/monitors/delete-monitor-api.asciidoc
@@ -8,9 +8,9 @@ Deletes one or more monitors from the Synthetics app.
 
 === {api-request-title}
 
-`DELETE <kibana host>:<port>/api/synthetics/monitors`
+`DELETE <kibana host>:<port>/api/synthetics/monitors/<config_id>`
 
-`DELETE <kibana host>:<port>/s/<space_id>/api/synthetics/monitors`
+`DELETE <kibana host>:<port>/s/<space_id>/api/synthetics/monitors/<config_id>`
 
 === {api-prereq-title}
 
@@ -20,6 +20,26 @@ You must have `all` privileges for the *Synthetics* feature in the *{observabili
 You must have `all` privileges for the *Synthetics* feature in the *{observability}* section of the
 <<kibana-feature-privileges,{kib} feature privileges>>.
 
+
+[[delete-monitor-api-path-params]]
+=== {api-path-parms-title}
+
+`config_id`::
+(Required, string) The ID of the monitor that you want to delete.
+
+
+Here is an example of a DELETE request to delete a monitor by ID:
+
+[source,sh]
+--------------------------------------------------
+DELETE /api/synthetics/monitors/monitor1-id
+--------------------------------------------------
+
+==== Bulk Delete Monitors
+
+You can delete multiple monitors by sending a list of config ids to a DELETE request to the `/api/synthetics/monitors` endpoint.
+
+
 [[monitors-delete-request-body]]
 ==== Request Body
 

diff --git a/...public/apps/synthetics/components/monitors_page/management/monitor_list_table/columns.tsx b/...public/apps/synthetics/components/monitors_page/management/monitor_list_table/columns.tsx
@@ -200,7 +200,7 @@ export function useMonitorListColumns({
         },
         {
           'data-test-subj': 'syntheticsMonitorCopyAction',
-          isPrimary: true,
+          isPrimary: false,
           name: (fields) => (
             <NoPermissionsTooltip
               canEditSynthetics={canEditSynthetics}

diff --git a/...k/plugins/observability_solution/synthetics/server/routes/monitor_cruds/delete_monitor.ts b/...k/plugins/observability_solution/synthetics/server/routes/monitor_cruds/delete_monitor.ts
@@ -29,30 +29,36 @@ import { formatSecrets, normalizeSecrets } from '../../synthetics_service/utils/
 
 export const deleteSyntheticsMonitorRoute: SyntheticsRestApiRouteFactory<
   DeleteParamsResponse[],
-  Record<string, any>,
-  Record<string, any>,
+  Record<string, string>,
+  Record<string, string>,
   { ids: string[] }
 > = () => ({
   method: 'DELETE',
-  path: SYNTHETICS_API_URLS.SYNTHETICS_MONITORS,
+  path: SYNTHETICS_API_URLS.SYNTHETICS_MONITORS + '/{id?}',
   validate: {},
   validation: {
     request: {
-      body: schema.object({
-        ids: schema.arrayOf(schema.string(), {
-          minSize: 1,
-        }),
+      body: schema.nullable(
+        schema.object({
+          ids: schema.arrayOf(schema.string(), {
+            minSize: 1,
+          }),
+        })
+      ),
+      params: schema.object({
+        id: schema.maybe(schema.string()),
       }),
     },
   },
   handler: async (routeContext): Promise<any> => {
     const { request, response } = routeContext;
 
-    const { ids } = request.body;
+    const { ids } = request.body || {};
+    const { id: queryId } = request.params;
 
     const result: Array<{ id: string; deleted: boolean; error?: string }> = [];
 
-    await pMap(ids, async (id) => {
+    await pMap([...(ids ?? []), ...(queryId ? [queryId] : [])], async (id) => {
       try {
         const { errors, res } = await deleteMonitor({
           routeContext,