Umbraco.Cms.Integrations icon indicating copy to clipboard operation
Umbraco.Cms.Integrations copied to clipboard
verified publisher icon umbraco

42318 V14 Integration: Zapier

Open dtng95 opened this issue 1 year ago • 3 comments

  • Like the PR for Shopify integration, I will update this PR daily
  • The first commit was to create management API for Zapier, and also the structure of the project.

dtng95 avatar Jul 25 '24 15:07 dtng95

@acoumb Do we need to update the appsettings file in V14 test site, in order to have the settings for Zapier, like we did for Algolia and Shopify (only the settings, no values).

dtng95 avatar Jul 25 '24 16:07 dtng95

Sure, you can do that.

acoumb avatar Jul 25 '24 16:07 acoumb

Hi @dtng95 ,

Implementation looks good, I made a couple of updates and cleaned up a little bit as detailed below:

  • broke down controllers into specific API endpoints to align with the other products. As a result, I've prepared a new major version of the Zapier Umbraco app to match the new endpoints.
  • updated client API resources
  • removed this line of code options.OperationFilter<BackOfficeSecurityRequirementsOperationFilter>(); as it generated an ArgumentException: An item with the same key has already been added. Key: 401 on my side. Could not figure out why.
  • cleaned up components UI.

Could you have a try at it with the new updates and see if it works alright for you as well? (use version 3.0.0 of the Zapier Umbraco app).

Thanks for the great work, Adrian

acoumb avatar Aug 05 '24 06:08 acoumb