Skip to content

V13: Add webhook information to detailed telemetry#16060

Merged
Zeegaan merged 6 commits intov13/devfrom
v13/feature/telemetry-update
Apr 16, 2024
Merged

V13: Add webhook information to detailed telemetry#16060
Zeegaan merged 6 commits intov13/devfrom
v13/feature/telemetry-update

Conversation

@nikolajlauridsen
Copy link
Contributor

Adds webhook information to detailed telemetry, a sample payload looks like the following:

        {
            "name": "WebhookCount_Total",
            "data": 3
        },
        {
            "name": "WebhookCount_Umbraco.ContentDelete",
            "data": 1
        },
        {
            "name": "WebhookCount_Umbraco.ContentPublish",
            "data": 1
        },
        {
            "name": "WebhookCount_Umbraco.ContentUnpublish",
            "data": 1
        },
        {
            "name": "WebhookCount_Umbraco.MediaDelete",
            "data": 1
        },
        {
            "name": "WebhookCount_Umbraco.MediaSave",
            "data": 1
        },
        {
            "name": "WebhookCount_CustomEvent",
            "data": 1
        },
        {
            "name": "WebhookCount_CustomHeaders",
            "data": 1
        }

Testing

Ensure that the payload is accurate, then can be achieved by setting a breakpoint in ReportSiteJob and ensure that the contents of telemetryReportData is as expected

@Zeegaan
Copy link
Member

Zeegaan commented Apr 16, 2024

Looks good, tests good 🚀

@Zeegaan Zeegaan merged commit 096991a into v13/dev Apr 16, 2024
@Zeegaan Zeegaan deleted the v13/feature/telemetry-update branch April 16, 2024 12:36
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.

2 participants