Skip to content

Feat/webhook names - #7

Open
tehno-for wants to merge 4 commits into
version-two:mainfrom
tehno-for:feat/webhook-names
Open

tehno-for wants to merge 4 commits into
version-two:mainfrom
tehno-for:feat/webhook-names

Conversation

@tehno-for

Copy link
Copy Markdown

Add a Name field to the webhook after it is created. It should appear as {destination} Webhook (****) - Name

tehno-for and others added 4 commits August 31, 2026 15:56
Accept Power Platform (`*.environment.api.powerplatform.com`) and Logic
Apps hosts, require a host label before `webhook.office.com` instead of
matching any domain ending in it, and scope the legacy Outlook form to
`/webhook/`. The placeholder now shows the Power Platform form too.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Webhook lists rendered every entry as a masked "Slack Webhook (********)"
placeholder, so several webhooks on the same platform were impossible to
tell apart once created.

WebhookConfiguration gains a nullable `name`, persisted as <name> in the
project settings XML (omitted when unset, so existing configuration and
backups load unchanged) and settable from the DSL via `webhook.name`. The
name is shown in place of the masked label on the project and build
configuration tabs, in the inherited/versioned-settings list, in the
client-side refreshed list, and on the admin overview cards; URLs stay
masked everywhere.

The name is display-only: identity, deduplication, inheritance and local
disabling still key off the URL. UI-entered names are capped at 100
characters, and the name is escaped at every render site, including the
JS list renderer, which previously had no escaping because it only ever
rendered server-controlled strings.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
The name replaced the "Slack Webhook (********)" label rather than
complementing it, so a named webhook no longer showed which platform it
targeted. Rows now render "Slack Webhook (********) - name" (and the same
for Teams and Discord), keeping the name optional.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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.

1 participant