Skip to content

Conversation

NguyenThuyLan
Copy link
Contributor

@NguyenThuyLan NguyenThuyLan commented Sep 30, 2025

πŸ“‹ Description

  • This PR rewrites the Property Actions documentation:

  • Clearer introduction with beginner-friendly context.

  • Step-by-step structure (concept β†’ UI β†’ registration β†’ implementation).

  • Improved code example with a practical use case.

πŸ“Ž Related Issues (if applicable)

#7452

βœ… Contributor Checklist

I've followed the Umbraco Documentation Style Guide and can confirm that:

  • Code blocks are correctly formatted.
  • Sentences are short and clear (preferably under 25 words).
  • Passive voice and first-person language (β€œwe”, β€œI”) are avoided.
  • Relevant pages are linked.
  • All links work and point to the correct resources.
  • Screenshots or diagrams are included if useful.
  • Any code examples or instructions have been tested.
  • Typos, broken links, and broken images are fixed.

Product & Version (if relevant)

Deadline (if relevant)

πŸ“š Helpful Resources

Copy link

The Link Checker found broken links in your PR.
Please review the following list:

Broken links found in: 16/umbraco-cms/customizing/property-editors/property-actions.md

βš“ Anchor not found β†’ 16/.gitbook/assets/property-actions-blocklist.png | Cannot find fileFull Github Actions output

Copy link
Contributor

@AndyButland AndyButland left a comment

Choose a reason for hiding this comment

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

Looks good to me @LanThuyNguyen. I've left a few inline comments for you to consider and see what you think.

Copy link
Contributor

@AndyButland AndyButland left a comment

Choose a reason for hiding this comment

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

Looks good to me now @NguyenThuyLan. Thanks. I'll leave this for @eshanrnh to do a final check and merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants