Skip to content

Conversation

@Migaroez
Copy link
Contributor

Prerequisites

Fixs #20272

Description

This a non breaking attempt to fix the above issue. It might not be the ideal solution, so I am open to all feedback.

@AndyButland AndyButland changed the title Fix : OutputExpansionStrategy for webhooks if DeliveryApi is not enabled Webhooks: Register OutputExpansionStrategy for webhooks if Delivery API is not enabled Oct 20, 2025
Copy link
Contributor

@AndyButland AndyButland left a comment

Choose a reason for hiding this comment

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

Looks reasonable to me @Migaroez. I've tested to replicate the original issue and confirmed with this code in place I don't see the problem, and the webhook is fired as expected. I left a few comments inline to consider, but other than that, seems good to me.

@AndyButland AndyButland merged commit 62c1d44 into main Oct 22, 2025
24 of 25 checks passed
@AndyButland AndyButland deleted the v16/bugfix/webhook-OutputExpansionStrategy branch October 22, 2025 11:46
@AndyButland AndyButland changed the title Webhooks: Register OutputExpansionStrategy for webhooks if Delivery API is not enabled Webhooks: Register OutputExpansionStrategy for webhooks if Delivery API is not enabled (closes #20272) Oct 22, 2025
AndyButland pushed a commit that referenced this pull request Oct 22, 2025
…PI is not enabled (#20559)

* Register slimmed down OutputExpansionStrategy for webhooks if deliveryapi is not enabled

* PR review comment resolution
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.

2 participants