Skip to content

Conversation

acoumb
Copy link
Contributor

@acoumb acoumb commented Sep 2, 2024

Description

Updated docs for the following integrations:

  • Umbraco.Cms.Integrations.Automation.Zapier 2.0.0
  • Umbraco.Forms.Integrations.Automation.Zapier 4.0.0

Type of suggestion

  • Typo/grammar fix
  • Updated outdated content
  • New content
  • Updates related to a new version
  • Other

Product & version (if relevant)

  • Umbraco.Cms.Integrations.Automation.Zapier 2.0.0
  • Umbraco.Forms.Integrations.Automation.Zapier 4.0.0

Deadline (if relevant)

Anytime

Copy link
Contributor

@sofietoft sofietoft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've added a couple of suggestions regarding the links.

I think it's a shame that this info is added to the bottom of the article.
If it's very important to use a specific version of Zapier for a specific Umbraco version, would it make sense to move this section to below the table with the "Minimum versions requirement" section further up in the articles?

@acoumb
Copy link
Contributor Author

acoumb commented Sep 3, 2024

It makes sense, although I do not expect a high number of installs of Zapier on Umbraco 14.
I've moved the section up below versioning.

Thanks @sofietoft

@sofietoft
Copy link
Contributor

Perfect. Thanks! 🙌

I'll get it merged.

@sofietoft sofietoft merged commit b7b5378 into umbraco:main Sep 3, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants