Skip to content

Conversation

vahidkay-meta
Copy link
Contributor

Throttle API call using transient to avoid recursive calls to wpcom

Description

Please include a summary of the changes and the related issue.
Please also include relevant motivation and context.
List any dependencies that are required for this change.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • I have commented my code, particularly in hard-to-understand areas.
  • I have confirmed that my changes do not introduce any new PHPCS warnings or errors.
  • I have checked plugin debug logs that my changes do not introduce any new PHP warnings or FATAL errors.
  • I followed general Pull Request best practices. Meta employees to follow this wiki.
  • I have added tests (if necessary) and all the new and existing unit tests pass locally with my changes.
  • I have completed dogfooding and QA testing, or I have conducted thorough due diligence to ensure that it does not break existing functionality.
  • I have updated or requested update to plugin documentations (if necessary).

Changelog entry

Fix: Throttle API call using transient to avoid recursive calls to wpcom

Test Plan

This should be tested manually.

@meta-cla meta-cla bot added the cla signed label Aug 11, 2025
Copy link
Collaborator

@iodic iodic left a comment

Choose a reason for hiding this comment

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

Looking good, thanks for moving the transient setting call above the option update! 🙈

@facebook-github-bot
Copy link
Contributor

@vahidkay-meta has imported this pull request. If you are a Meta employee, you can view this in D79994875.

@facebook-github-bot
Copy link
Contributor

@vahidkay-meta has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@vahidkay-meta has imported this pull request. If you are a Meta employee, you can view this in D79994875.

@facebook-github-bot
Copy link
Contributor

@vahidkay-meta has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@vahidkay-meta has imported this pull request. If you are a Meta employee, you can view this in D79994875.

@facebook-github-bot
Copy link
Contributor

@vahidkay-meta merged this pull request in 340eb36.

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.

4 participants