Merge pull request #266 from umbraco/v16/zapier

V16/zapier

Author: acoumb

src/Umbraco.Cms.Integrations.Automation.Zapier/Api/Configuration/ZapierOperationIdHandler.cs

@@ -0,0 +1,18 @@
+using Asp.Versioning;
+using Microsoft.AspNetCore.Mvc.ApiExplorer;
+using Microsoft.AspNetCore.Mvc.Controllers;
+using Microsoft.Extensions.Options;
+using Umbraco.Cms.Api.Common.OpenApi;
+
+namespace Umbraco.Cms.Integrations.Automation.Zapier.Api.Configuration
+{
+    internal class ZapierOperationIdHandler : OperationIdHandler
+    {
+        public ZapierOperationIdHandler(IOptions<ApiVersioningOptions> apiVersioningOptions) : base(apiVersioningOptions)
+        {
+        }
+
+        protected override bool CanHandle(ApiDescription apiDescription, ControllerActionDescriptor controllerActionDescriptor)
+            => controllerActionDescriptor.ControllerTypeInfo.Namespace?.StartsWith("Umbraco.Cms.Integrations.Automation.") is true;
+    }
+}

src/Umbraco.Cms.Integrations.Automation.Zapier/Api/Management/Controllers/GetContentByTypeController.cs

@@ -39,9 +39,8 @@ public GetContentByTypeController(
             _umbracoHelper = umbracoHelper;
         }
 
-        [HttpGet("content-type/{alias}/content")]
+        [HttpGet("content-type/{alias}/content", Name = Constants.OperationIdentifiers.GetContentByType)]
         [ProducesResponseType(typeof(List<Dictionary<string, string>>), StatusCodes.Status200OK)]
-        [ProducesResponseType(StatusCodes.Status401Unauthorized)]
         public IActionResult GetContentByContentType(string alias)
         {
             if (!IsAccessValid())

src/Umbraco.Cms.Integrations.Automation.Zapier/Api/Management/Controllers/GetContentTypesController.cs

@@ -25,7 +25,6 @@ public GetContentTypesController(IOptions<ZapierSettings> options, IContentTypeS
 
         [HttpGet("content-types")]
         [ProducesResponseType(typeof(IEnumerable<ContentTypeDto>), StatusCodes.Status200OK)]
-        [ProducesResponseType(StatusCodes.Status401Unauthorized)]
         public IActionResult GetContentTypes()
         {
             if (!IsAccessValid()) 

src/Umbraco.Cms.Integrations.Automation.Zapier/Api/Management/Controllers/GetSubscriptionHookController.cs

@@ -20,7 +20,7 @@ public GetSubscriptionHookController(
             ZapierSubscriptionHookService zapierSubscriptionHookService)
             : base(options, userValidationService) => _zapierSubscriptionHookService = zapierSubscriptionHookService;
 
-        [HttpGet("subscription-hooks")]
+        [HttpGet("subscription-hooks", Name = Constants.OperationIdentifiers.GetSubscriptionHooks)]
         [ProducesResponseType(typeof(IEnumerable<SubscriptionDto>), StatusCodes.Status200OK)]
         public IActionResult GetAll() => Ok(_zapierSubscriptionHookService.GetAll());
     }

src/Umbraco.Cms.Integrations.Automation.Zapier/Api/Management/Controllers/UpdateSubscriptionHookController.cs

@@ -25,8 +25,7 @@ public UpdateSubscriptionHookController(IOptions<ZapierSettings> options,
             _zapierSubscriptionHookService = zapierSubscriptionHookService;
         }
 
-        [HttpPost("subscription")]
-        [ProducesResponseType(StatusCodes.Status401Unauthorized)]
+        [HttpPost("subscription", Name = Constants.OperationIdentifiers.UpdateSubscription)]
         [ProducesResponseType(StatusCodes.Status400BadRequest)]
         [ProducesResponseType(typeof(bool), StatusCodes.Status200OK)]
         public IActionResult UpdatePreferences([FromBody] SubscriptionDto dto)

src/Umbraco.Cms.Integrations.Automation.Zapier/Client/generated/client.gen.ts

@@ -0,0 +1,19 @@
+// This file is auto-generated by @hey-api/openapi-ts
+
+import type { ClientOptions } from './types.gen';
+import { type Config, type ClientOptions as DefaultClientOptions, createClient, createConfig } from '@hey-api/client-fetch';
+
+/**
+ * The `createClientConfig()` function will be called on client initialization
+ * and the returned object will become the client's initial configuration.
+ *
+ * You may want to initialize your client this way instead of calling
+ * `setConfig()`. This is useful for example if you're using Next.js
+ * to ensure your client always has the correct values.
+ */
+export type CreateClientConfig<T extends DefaultClientOptions = ClientOptions> = (override?: Config<DefaultClientOptions & T>) => Config<Required<DefaultClientOptions> & T>;
+
+export const client = createClient(createConfig<ClientOptions>({
+    baseUrl: 'http://localhost:30450',
+    throwOnError: true
+}));