Change Details

This hook was introduced in the past to support some advanced experimentation needs of the Growth Team in our bespoke experimentation setup. However, with TestKitchen having replaced that custom system, this hook is no longer needed. Given that it is an undocumented (T383702) and complex hook in a complex feature, it seems simpler to drop it again instead of keeping it around and paying the associated maintenance overhead when we don't actually have a use for it. **Acceptance criteria:** * [ ] the `onConditionalDefaultOptionsAddCondition` hook has been removed from core again

This hook was introduced in the past to support some advanced experimentation needs of the Growth Team in our bespoke experimentation setup. However, with TestKitchen having replaced that custom system, this hook is no longer needed. Given that it is an undocumented (T383702) and complex hook in a complex feature, it seems simpler to drop it again instead of keeping it around and paying the associated maintenance overhead when we don't actually have a use for it. **Acceptance criteria:** [x] All known users removed [x] The hook is deprecated (in MW 1.46) [ ] the `ConditionalDefaultOptionsAddCondition` hook has been removed from core again (in MW 1.47+)

This hook was introduced in the past to support some advanced experimentation needs of the Growth Team in our bespoke experimentation setup. However, with TestKitchen having replaced that custom system, this hook is no longer needed. Given that it is an undocumented (T383702) and complex hook in a complex feature, it seems simpler to drop it again instead of keeping it around and paying the associated maintenance overhead when we don't actually have a use for it. **Acceptance criteria:** * [ ] the `on[x] All known users removed [x] The hook is deprecated (in MW 1.46) [ ] the `ConditionalDefaultOptionsAddCondition` hook has been removed from core again (in MW 1.47+)