Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

58757 - Improve performance of the widget content #4819

Open
wants to merge 3 commits into
base: trunk
Choose a base branch
from
Open

58757 - Improve performance of the widget content #4819

wants to merge 3 commits into from

Conversation

nirav7707
Copy link

Trac Ticke

  • PR used to add the _prime_post_caches( $attachments ); to cache the posts before iterate to enhance performance.
@nirav7707
Add: function _prime_post_caches to improve the performance by caching
2ecd08a 
_prime_post_caches to to cache the attachments if it's not in cache, by
caching the posts will improve the performance the code behaviour remains
same.
@spacedmonkey spacedmonkey self-requested a review July 10, 2023 11:57
@mukeshpanchal27 mukeshpanchal27 self-requested a review July 11, 2023 15:21
spacedmonkey
spacedmonkey approved these changes Aug 31, 2023
View reviewed changes
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.

This looks good to commit to me
@nirav7707 @mukeshpanchal27
Apply suggestions from code review
43445ab 
Co-authored-by: Mukesh Panchal <mukeshpanchal27@users.noreply.github.com>
@mukeshpanchal27 mukeshpanchal27 mentioned this pull request Aug 31, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants
@nirav7707 @spacedmonkey @mukeshpanchal27