Skip to content
This repository was archived by the owner on Nov 6, 2025. It is now read-only.

Conversation

@manutdkid77
Copy link
Contributor

@manutdkid77 manutdkid77 commented Oct 30, 2024

Description

This fixes 16388

Replicated on 15.1.0-rc1.preview.16.

Types of changes

The URL is a mandatory field for a webhook, so I've set it as required. Additionally I have also set it to autofocus on load.

Screenshots

2024-10-30.20-46-00.mp4

removed redundant `true` value from the `mandatory` attribute
Copy link
Member

@leekelleher leekelleher left a comment

Choose a reason for hiding this comment

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

Super, thanks @manutdkid77! 🙏

FYI, I'll cherry-pick this for v15.0.0-rc3.

@leekelleher leekelleher merged commit b257756 into umbraco:main Nov 4, 2024
8 checks passed
leekelleher pushed a commit that referenced this pull request Nov 4, 2024
* Feature: Set the URL field as required and focus on it

* Added mandatory asterix to webhook url label

---------

Co-authored-by: Nathaniel Nunes <[email protected]>
(cherry picked from commit b257756)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Webhook - No client side validation

3 participants