Skip to content

Conversation

@SH4LIN
Copy link

@SH4LIN SH4LIN commented May 23, 2023

  • Added the _deprecate_function() inside the wp_queue_comments_for_comment_meta_lazyload() function. Updated the DocBlock for the same function.
  • Updated the DocBlock of WP_Query::lazyload_term_meta( $check, $term_id ) method.

Trac ticket: https://core.trac.wordpress.org/ticket/58301


This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.

SH4LIN added 3 commits May 23, 2023 12:10
…e_function() inside the wp_queue_comments_for_comment_meta_lazyload() function.

- Updated the DocBlock of WP_Query::lazyload_comment_meta() function.
@spacedmonkey spacedmonkey self-requested a review May 23, 2023 15:14
@SH4LIN SH4LIN requested a review from spacedmonkey May 23, 2023 16:07
Copy link
Member

@spacedmonkey spacedmonkey left a comment

Choose a reason for hiding this comment

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

One more change and I will commit.

Copy link
Member

@spacedmonkey spacedmonkey left a comment

Choose a reason for hiding this comment

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

Approved. Well done @SH4LIN 🎉

Copy link
Contributor

@peterwilsoncc peterwilsoncc left a comment

Choose a reason for hiding this comment

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

I've added a few very minor suggestions inline.

LGTM code wise though.

@SH4LIN SH4LIN requested a review from peterwilsoncc May 25, 2023 02:54
@spacedmonkey
Copy link
Member

Committed 48683a2

@SH4LIN SH4LIN deleted the deprecate-wp-queue-comments-for-comment-meta-lazyload-function/58301 branch May 31, 2023 06:17
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.

3 participants