Skip to content

Bump Wordpress Version minimum requirement 6.6#1312

Merged
jeffpaul merged 3 commits intodevelopfrom
Bump-require-at-least-wordpress-version-6.6
May 12, 2025
Merged

Bump Wordpress Version minimum requirement 6.6#1312
jeffpaul merged 3 commits intodevelopfrom
Bump-require-at-least-wordpress-version-6.6

Conversation

@Sourabh208
Copy link
Copy Markdown
Contributor

Description of the Change

Closes #1308

How to test the Change

Follow the critical flows to test the PR

Changelog Entry

Changed - Bump WordPress "require at least" version 6.6

@ankitguptaindia @jeffpaul @vikrampm1 I've created this PR to address the closure of issue #1308 . In this update, I've only modified the minimum required WordPress version to 6.6.

Minimum Requirement Version 6.6
@Sourabh208 Sourabh208 requested review from a team and jeffpaul as code owners April 22, 2025 09:16
@Sourabh208 Sourabh208 requested review from vikrampm1 and removed request for a team and jeffpaul April 22, 2025 09:16
@github-actions github-actions bot added this to the 2.2.0 milestone Apr 22, 2025
@github-actions github-actions bot added the needs:feedback This requires reporter feedback to better understand the request. label Apr 22, 2025
@github-actions
Copy link
Copy Markdown

@Sourabh208 thanks for the PR! Could you please fill out the PR template with description, changelog, and credits information so that we can properly review and merge this?

@jeffpaul
Copy link
Copy Markdown
Member

e2e tests are failing, @10up/open-source-practice will want to check what's happening there before merging this in

vikrampm1
vikrampm1 previously approved these changes Apr 25, 2025
@peterwilsoncc peterwilsoncc force-pushed the Bump-require-at-least-wordpress-version-6.6 branch from a607947 to 39071ad Compare May 7, 2025 05:51
@peterwilsoncc
Copy link
Copy Markdown
Collaborator

@jeffpaul The failed tests were false negatives caused by some DOM changes with regard to how the media library was injected in to the page.

39071ad includes a fix for this by ensuring that the copy of the media library the tests interact with is the visible.

I've pushed a related change to the artifact names so that failures from multiple test runs are included when applicable.

@jeffpaul jeffpaul merged commit d363e1f into develop May 12, 2025
17 of 18 checks passed
@jeffpaul jeffpaul deleted the Bump-require-at-least-wordpress-version-6.6 branch May 12, 2025 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs:feedback This requires reporter feedback to better understand the request.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The plugin hasn't been tested with an upcoming version of WordPress

4 participants