Skip to content

Conversation

@t-hamano
Copy link
Contributor

See this comment: https://github.com/WordPress/wordpress-develop/pull/5922/files#r1470113401

What?

This PR adds @since notation to the filter hook for the random order that was added to the Gallery Block in #57477.

Why?

As mentioned in #52720, we should have added this notation to the block-library PHP files.

@t-hamano t-hamano added [Type] Code Quality Issues or PRs that relate to code quality [Block] Gallery Affects the Gallery Block - used to display groups of images labels Jan 30, 2024
@t-hamano t-hamano requested a review from geriux as a code owner January 30, 2024 13:23
@github-actions
Copy link

github-actions bot commented Feb 2, 2024

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Core SVN

Core Committers: Use this line as a base for the props when committing in SVN:

Props wildworks.

GitHub Merge commits

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: t-hamano <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@t-hamano
Copy link
Contributor Author

t-hamano commented Feb 7, 2024

Update: This filter hook was removed by #58733. I'm closing this PR as the notation is no longer needed.

@t-hamano t-hamano closed this Feb 7, 2024
@johnbillion johnbillion deleted the gallery/add-since-notation branch February 10, 2025 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Block] Gallery Affects the Gallery Block - used to display groups of images [Type] Code Quality Issues or PRs that relate to code quality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants