Skip to content

Make preview check for delivery API content case insensitive#18731

Merged
kjac merged 1 commit intov13/devfrom
v13/bugfix/delivery-api-request-preview-case-insensitive
Mar 20, 2025
Merged

Make preview check for delivery API content case insensitive#18731
kjac merged 1 commit intov13/devfrom
v13/bugfix/delivery-api-request-preview-case-insensitive

Conversation

@AndyButland
Copy link
Copy Markdown
Contributor

Prerequisites

  • I have added steps to test this contribution in the description below

Resolves #18730

Description

This PR makes the preview check for delivery API content case insensitive, thus supporting true or True as is required in the linked issue.

To Test:

  • See "Steps to Reproduce" in the linked issue.

@AndyButland AndyButland requested a review from kjac March 19, 2025 14:04
Copy link
Copy Markdown
Contributor

@kjac kjac left a comment

Choose a reason for hiding this comment

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

Looks great 💪 extra points for the test 🤩

@kjac kjac merged commit eb91f4f into v13/dev Mar 20, 2025
20 checks passed
@kjac kjac deleted the v13/bugfix/delivery-api-request-preview-case-insensitive branch March 20, 2025 05:53
@kjac
Copy link
Copy Markdown
Contributor

kjac commented Mar 20, 2025

Cherry-picked for V13.8 and V15.latest

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