-
Notifications
You must be signed in to change notification settings - Fork 813
Update Engage v17 examples & requirements #7678
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, @cornehoskam 🙌I have added a few suggestions based on the style guide and to fix Vale errors.
For future maintenance, I have removed references to 17. Since these articles are already specific to version 17, it is an extra task to keep updating them for future releases.
Please let me know if it doesn't make sense.
📋 Description
The Umbraco Engage documentation contained a lot of broken examples and missing information regarding requirement. With the help of Claude everything has now been brought up-to-date with the latest and greatest code on Engage v17 😄
📎 Related Issues (if applicable)
✅ Contributor Checklist
I've followed the Umbraco Documentation Style Guide and can confirm that:
Product & Version (if relevant)
Umbraco Engage v17
Deadline (if relevant)
📚 Helpful Resources