Skip to content

Conversation

@m1r0
Copy link
Member

@m1r0 m1r0 commented Dec 11, 2025

Part of #7873

Proposed Changes

  • Related to this Gutenberg change (and the following fix), patterns are now required to be wrapped in a Query block. This PR fixes that for the Course List pattern (course-grid-with-background).

Testing Instructions

  1. Check-out this Gutenberg PR.
  2. Test the Course List block and make sure the Course List pattern is working as expected.

Pre-Merge Checklist

  • PR title and description contain sufficient detail and accurately describe the changes
  • Adheres to coding standards (PHP, JavaScript, CSS, HTML)
  • All strings are translatable (without concatenation, handles plurals)
  • Follows our naming conventions (P6rkRX-4oA-p2)
  • Hooks (p6rkRX-1uS-p2) and functions are documented
  • New UIs are responsive and use a mobile-first approach
  • Code is tested on the minimum supported PHP and WordPress versions

@m1r0 m1r0 self-assigned this Dec 11, 2025
Copilot AI review requested due to automatic review settings December 11, 2025 10:48
@m1r0 m1r0 added this to the 4.25.2 milestone Dec 11, 2025
@m1r0
Copy link
Member Author

m1r0 commented Dec 11, 2025

It's normal for the E2E tests to be failing because the suite is not using the latest Gutenberg.

@m1r0 m1r0 merged commit 74540f7 into trunk Dec 11, 2025
28 of 30 checks passed
@m1r0 m1r0 deleted the fix/course-list-patterns branch December 11, 2025 10:59
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