Skip to content
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

Feature: Set the URL field as required and focus on it #2500

Merged

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 <n.nunes@clerkswell.com>
(cherry picked from commit b257756)
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.

Webhook - No client side validation
2 participants