Skip to content

Adding details and examples for Redirect URL notifications (18.1) - #8120

Merged
eshanrnh merged 4 commits into
mainfrom
v18/feature/redirect-notifications
Jul 23, 2026
Merged

Adding details and examples for Redirect URL notifications (18.1)#8120
eshanrnh merged 4 commits into
mainfrom
v18/feature/redirect-notifications

Conversation

@NillasKA

@NillasKA NillasKA commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

📋 Description

A new feature has been introduced, which adds notifications to saving, saved, deleting and deleted redirects. These docs explain how to use these notifications and provide concrete examples.

📎 Related Issues (if applicable)

umbraco/Umbraco-CMS#22985

✅ 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)

CMS v18.1

Deadline (if relevant)

23rd of July when the RC is due.

📚 Helpful Resources

@github-actions

github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown

Thanks for opening this PR! 🎉

Quick question: Does this PR include any AI-generated content?

AI tools are absolutely welcome here! We only ask that you follow our AI contribution guidelines to help keep the documentation consistent and trustworthy.

Drop a quick comment to let us know!

@NillasKA

NillasKA commented Jun 3, 2026

Copy link
Copy Markdown
Contributor Author

A thing to note here is my IDE does not translate the GitBook syntax like {% code title="PreventRedirectCreationHandler.cs" %} - I tried following the pattern established in other files, but please correct me if it's wrong.

@NillasKA

NillasKA commented Jun 3, 2026

Copy link
Copy Markdown
Contributor Author

The Link checker fails on this, because the build has not been released yet, therefore the links do not exist yet. It should however be correct, will have to double check when we release this

@AndyButland AndyButland left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approving with just one comment to consider, otherwise all looks really good.

@github-actions

github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown

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

Broken links found in: 18/umbraco-cms/extend-your-project/server-side-extensions/notifications/README.md

❌ 404 Not Found → https://apidocs.umbraco.com/v18/csharp/api/Umbraco.Cms.Core.Notifications.RedirectUrlDeletedNotification.html | Rejected status code: 404 Not Found (configurable with "accept" option)

❌ 404 Not Found → https://apidocs.umbraco.com/v18/csharp/api/Umbraco.Cms.Core.Notifications.RedirectUrlDeletingNotification.html | Rejected status code: 404 Not Found (configurable with "accept" option)

❌ 404 Not Found → https://apidocs.umbraco.com/v18/csharp/api/Umbraco.Cms.Core.Notifications.RedirectUrlSavedNotification.html | Rejected status code: 404 Not Found (configurable with "accept" option)

❌ 404 Not Found → https://apidocs.umbraco.com/v18/csharp/api/Umbraco.Cms.Core.Notifications.RedirectUrlSavingNotification.html | Rejected status code: 404 Not Found (configurable with "accept" option)

@eshanrnh eshanrnh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great 🙌

Comment thread 18/umbraco-cms/extend-your-project/server-side-extensions/notifications/README.md Outdated
@eshanrnh
eshanrnh merged commit e052aae into main Jul 23, 2026
61 of 63 checks passed
@eshanrnh
eshanrnh deleted the v18/feature/redirect-notifications branch July 23, 2026 09:12
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.

4 participants