Changes the API URL to use the most current one (adds /hosting to the REST API url)#88
Changes the API URL to use the most current one (adds /hosting to the REST API url)#88
Conversation
WalkthroughThe changes update three functions responsible for interacting with WordPress.com site code deployment APIs by modifying the endpoint paths used in their HTTP requests. The affected functions now use the 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🧰 Additional context used🧬 Code Graph Analysis (1)includes/functions-wpcom.php (1)
🔇 Additional comments (3)
✨ Finishing Touches
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
WE're having issues with the following command not working:
team51 wpcom:connect-site-repository 242967371 psychedelic-safety --branch=developa 500 is thrown with an error about a bad request.This PR changes the API URL to take into account the wpcom URL: https://opengrok.a8c.com/source/search?project=wpcom&full=code-deployments&defs=&refs=&path=&hist=&type=&xrd=&nn=1 which has /hosting in the API URL.
https://developer.wordpress.com/docs/api/
https://developer.wordpress.com/docs/api/2/post/sites/$wpcom_site/hosting/code-deployments/
Summary by CodeRabbit