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.

NguyenThuyLan and others added 8 commits October 1, 2025 09:48
Co-authored-by: Andy Butland <abutland73@gmail.com>
Co-authored-by: Andy Butland <abutland73@gmail.com>
Co-authored-by: Andy Butland <abutland73@gmail.com>
Co-authored-by: Andy Butland <abutland73@gmail.com>
Co-authored-by: Andy Butland <abutland73@gmail.com>
Co-authored-by: Andy Butland <abutland73@gmail.com>
Co-authored-by: Andy Butland <abutland73@gmail.com>
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.

Copy link
Contributor

@eshanrnh eshanrnh left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, @NguyenThuyLan πŸ™Œ The PR looks great πŸ’ͺ

@eshanrnh eshanrnh merged commit 0c18a33 into main Oct 10, 2025
38 of 47 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

4 participants