Skip to content

Conversation

@leogermani
Copy link
Contributor

@leogermani leogermani commented Aug 3, 2020

This method was replaced with a more complete class in #16674

Its usage was replaced in

Now it's no longer used and should be deprecated, especially because we don't want to use JETPACK_MASTER_USER token anymore.

Changes proposed in this Pull Request:

  • Deprecates Jetpack::xmlrpc_async_call which is no longer used

Jetpack product discussion

n/a

Does this pull request change what data or activity we track or use?

No

Testing instructions:

  • Nothing specific to test, just make sure there are no warnings or notices anywhere
  • Any thoughts of external plugins/blocks that could be using it?

Proposed changelog entry for your changes:

  • Deprecates Jetpack::xmlrpc_async_call

@jetpackbot
Copy link
Collaborator

jetpackbot commented Aug 3, 2020

Thank you for the great PR description!

When this PR is ready for review, please apply the [Status] Needs Review label. If you are an a11n, please have someone from your team review the code if possible. The Jetpack team will also review this PR and merge it to be included in the next Jetpack release.

E2E results is available here (for debugging purposes): https://jetpack-e2e-dashboard.herokuapp.com/pr-16691

Scheduled Jetpack release: September 1, 2020.
Scheduled code freeze: August 25, 2020

Generated by 🚫 dangerJS against 5e342b3

@leogermani leogermani force-pushed the update/depreacate_xmlrpc_async_call branch from 1d800e5 to e4a6f48 Compare August 19, 2020 18:55
@leogermani leogermani force-pushed the update/depreacate_xmlrpc_async_call branch from e4a6f48 to 894aa2e Compare August 19, 2020 20:45
@leogermani leogermani changed the title Update/depreacate xmlrpc async call Depreacate xmlrpc async call Aug 19, 2020
@leogermani leogermani self-assigned this Aug 19, 2020
@leogermani leogermani marked this pull request as ready for review August 19, 2020 21:19
@leogermani leogermani added the [Status] Needs Review This PR is ready for review. label Aug 19, 2020
@leogermani leogermani added this to the 8.9 milestone Aug 19, 2020
@kbrown9 kbrown9 self-requested a review August 21, 2020 14:23
Copy link
Member

@kbrown9 kbrown9 left a comment

Choose a reason for hiding this comment

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

I tested this, and everything was fine.

I searched WPDirectory for other plugins using 'xmlrpc_async_call', and it found Jetpack and Slim Jetpack, which implements its own Jetpack::xmlrpc_async_call. So, I think it's fine to deprecate this. If another plugin is using it, the method still works.

I added a few comments below.

@kbrown9 kbrown9 added [Status] Needs Author Reply We need more details from you. This label will be auto-added until the PR meets all requirements. [Type] Janitorial and removed [Status] Needs Review This PR is ready for review. labels Aug 21, 2020
@jeherve jeherve removed this from the 8.9 milestone Aug 25, 2020
leogermani and others added 2 commits August 25, 2020 15:03
@leogermani leogermani added this to the 8.9 milestone Aug 25, 2020
@leogermani leogermani added [Status] Needs Review This PR is ready for review. [Status] Needs Cherry-Pick and removed [Status] Needs Author Reply We need more details from you. This label will be auto-added until the PR meets all requirements. labels Aug 25, 2020
@leogermani
Copy link
Contributor Author

Sorry I missed the code freeze. But accepted Kim's changes and I think this is good to go now!

Copy link
Member

@kbrown9 kbrown9 left a comment

Choose a reason for hiding this comment

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

LGTM!

@jeherve jeherve added [Package] Connection [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review This PR is ready for review. labels Aug 26, 2020
@jeherve jeherve merged commit c2d9c3e into master Aug 26, 2020
@jeherve jeherve deleted the update/depreacate_xmlrpc_async_call branch August 26, 2020 07:48
@matticbot matticbot removed the [Status] Ready to Merge Go ahead, you can push that green button! label Aug 26, 2020
jeherve pushed a commit that referenced this pull request Aug 26, 2020
@jeherve
Copy link
Member

jeherve commented Aug 26, 2020

Cherry-picked to branch-8.9 in 94c5a6a

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.

6 participants