Add jetpack endpoint for jetpack product data (#33095)

* changelog * Add barebones infrastructure for querying product data in the plugin * changelog * Fix product data api call * Fix incorrect function call * Update api_version * Update version number * Force wpcom endpoint * Update route setup * Remove useless alias * Remove logs * Remove backup specific endpoint * Fixup project versions * Fix composer lock files * changelog * Remove useless comment
Automattic · Sep 20, 2023 · b0c967a · b0c967a
1 parent 5797577
commit b0c967a
Show file tree

Hide file tree

Showing 33 changed files with 215 additions and 28 deletions.
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/projects/packages/my-jetpack/_inc/hooks/use-product-data/index.js b/projects/packages/my-jetpack/_inc/hooks/use-product-data/index.js
@@ -0,0 +1,14 @@
+import { useSelect } from '@wordpress/data';
+import { STORE_ID } from '../../state/store';
+
+/**
+ * React custom hook to get the product data
+ *
+ * @returns {object} product data
+ */
+export default function useProductData() {
+	return {
+		productData: useSelect( select => select( STORE_ID ).getProductData() ),
+		fetchingProductData: useSelect( select => select( STORE_ID ).isFetchingProductData() ),
+	};
+}
diff --git a/projects/packages/my-jetpack/_inc/state/actions.js b/projects/packages/my-jetpack/_inc/state/actions.js
@@ -21,6 +21,8 @@ const SET_CHAT_AVAILABILITY_IS_FETCHING = 'SET_CHAT_AVAILABILITY_IS_FETCHING';
 const SET_CHAT_AVAILABILITY = 'SET_CHAT_AVAILABILITY';
 const SET_CHAT_AUTHENTICATION_IS_FETCHING = 'SET_CHAT_AUTHENTICATION_IS_FETCHING';
 const SET_CHAT_AUTHENTICATION = 'SET_CHAT_AUTHENTICATION';
+const SET_PRODUCT_DATA_IS_FETCHING = 'SET_PRODUCT_DATA_IS_FETCHING';
+const SET_PRODUCT_DATA = 'SET_PRODUCT_DATA';
 
 const SET_GLOBAL_NOTICE = 'SET_GLOBAL_NOTICE';
 const CLEAN_GLOBAL_NOTICE = 'CLEAN_GLOBAL_NOTICE';
@@ -40,6 +42,10 @@ const setChatAuthenticationIsFetching = isFetching => {
 	return { type: SET_CHAT_AUTHENTICATION_IS_FETCHING, isFetching };
 };
 
+const setProductDataIsFetching = isFetching => {
+	return { type: SET_PRODUCT_DATA_IS_FETCHING, isFetching };
+};
+
 const fetchPurchases = () => {
 	return { type: FETCH_PURCHASES };
 };
@@ -70,6 +76,8 @@ const setAvailableLicenses = availableLicenses => {
 
 const setProduct = product => ( { type: SET_PRODUCT, product } );
 
+const setProductData = productData => ( { type: SET_PRODUCT_DATA, productData } );
+
 const setRequestProductError = ( productId, error ) => ( {
 	type: SET_PRODUCT_REQUEST_ERROR,
 	productId,
@@ -271,6 +279,8 @@ const actions = {
 	setPurchases,
 	setChatAvailability,
 	setChatAuthentication,
+	setProductDataIsFetching,
+	setProductData,
 	setAvailableLicensesIsFetching,
 	fetchAvailableLicenses,
 	setAvailableLicenses,
@@ -300,5 +310,7 @@ export {
 	SET_CHAT_AVAILABILITY_IS_FETCHING,
 	SET_CHAT_AUTHENTICATION,
 	SET_CHAT_AUTHENTICATION_IS_FETCHING,
+	SET_PRODUCT_DATA_IS_FETCHING,
+	SET_PRODUCT_DATA,
 	actions as default,
 };
diff --git a/projects/packages/my-jetpack/_inc/state/constants.js b/projects/packages/my-jetpack/_inc/state/constants.js
@@ -1,6 +1,7 @@
 const REST_API_NAMESPACE = 'my-jetpack/v1';
 export const REST_API_SITE_PURCHASES_ENDPOINT = `${ REST_API_NAMESPACE }/site/purchases`;
 export const REST_API_SITE_PRODUCTS_ENDPOINT = `${ REST_API_NAMESPACE }/site/products`;
+export const REST_API_SITE_PRODUCT_DATA_ENDPOINT = `${ REST_API_NAMESPACE }/site/product-data`;
 export const REST_API_CHAT_AVAILABILITY_ENDPOINT = `${ REST_API_NAMESPACE }/chat/availability`;
 export const REST_API_CHAT_AUTHENTICATION_ENDPOINT = `${ REST_API_NAMESPACE }/chat/authentication`;
 export const PRODUCTS_THAT_NEEDS_INITIAL_FETCH = [ 'scan' ];

diff --git a/projects/packages/my-jetpack/_inc/state/reducers.js b/projects/packages/my-jetpack/_inc/state/reducers.js
@@ -16,6 +16,8 @@ import {
 	CLEAN_GLOBAL_NOTICE,
 	SET_PRODUCT_STATS,
 	SET_IS_FETCHING_PRODUCT_STATS,
+	SET_PRODUCT_DATA_IS_FETCHING,
+	SET_PRODUCT_DATA,
 } from './actions';
 
 const products = ( state = {}, action ) => {
@@ -77,6 +79,25 @@ const products = ( state = {}, action ) => {
 	}
 };
 
+const productData = ( state = {}, action ) => {
+	switch ( action.type ) {
+		case SET_PRODUCT_DATA_IS_FETCHING:
+			return {
+				...state,
+				isFetching: action.isFetching,
+			};
+
+		case SET_PRODUCT_DATA:
+			return {
+				...state,
+				items: action?.productData || {},
+			};
+
+		default:
+			return state;
+	}
+};
+
 const purchases = ( state = {}, action ) => {
 	switch ( action.type ) {
 		case SET_PURCHASES_IS_FETCHING:
@@ -213,6 +234,7 @@ const stats = ( state = {}, action ) => {
 
 const reducers = combineReducers( {
 	products,
+	productData,
 	purchases,
 	chatAvailability,
 	chatAuthentication,

diff --git a/projects/packages/my-jetpack/_inc/state/resolvers.js b/projects/packages/my-jetpack/_inc/state/resolvers.js
@@ -13,6 +13,7 @@ import {
 	REST_API_SITE_PRODUCTS_ENDPOINT,
 	REST_API_CHAT_AVAILABILITY_ENDPOINT,
 	REST_API_CHAT_AUTHENTICATION_ENDPOINT,
+	REST_API_SITE_PRODUCT_DATA_ENDPOINT,
 	PRODUCTS_THAT_NEEDS_INITIAL_FETCH,
 } from './constants';
 import resolveProductStatsRequest from './stats-resolvers';
@@ -129,6 +130,19 @@ const myJetpackResolvers = {
 				dispatch.setAvailableLicensesIsFetching( false );
 			}
 		},
+
+	getProductData: () => {
+		return async ( { dispatch } ) => {
+			dispatch.setProductDataIsFetching( true );
+
+			try {
+				dispatch.setProductData( await apiFetch( { path: REST_API_SITE_PRODUCT_DATA_ENDPOINT } ) );
+				dispatch.setProductDataIsFetching( false );
+			} catch ( error ) {
+				dispatch.setProductDataIsFetching( false );
+			}
+		};
+	},
 };
 
 const getProductStats = {

diff --git a/projects/packages/my-jetpack/_inc/state/selectors.js b/projects/packages/my-jetpack/_inc/state/selectors.js
@@ -77,6 +77,11 @@ const productSelectors = {
 	getProductsThatRequiresUserConnection,
 };
 
+const productDataSelectors = {
+	getProductData: state => state.productData?.items || {},
+	isFetchingProductData: state => state.productData?.isFetching || false,
+};
+
 const purchasesSelectors = {
 	getPurchases: state => state.purchases?.items || [],
 	isRequestingPurchases: state => state.purchases?.isFetching || false,
@@ -134,6 +139,7 @@ const selectors = {
 	...purchasesSelectors,
 	...chatAvailabilitySelectors,
 	...chatAuthenticationSelectors,
+	...productDataSelectors,
 	...availableLicensesSelectors,
 	...noticeSelectors,
 	...pluginSelectors,

diff --git a/projects/packages/my-jetpack/changelog/add-jetpack-endpoint-for-jetpack-product-data b/projects/packages/my-jetpack/changelog/add-jetpack-endpoint-for-jetpack-product-data
@@ -0,0 +1,4 @@
+Significance: minor
+Type: added
+
+Add barebones infrastructure for querying jetpack product data
diff --git a/projects/packages/my-jetpack/composer.json b/projects/packages/my-jetpack/composer.json
@@ -74,7 +74,7 @@
 			"link-template": "https://github.com/Automattic/jetpack-my-jetpack/compare/${old}...${new}"
 		},
 		"branch-alias": {
-			"dev-trunk": "3.5.x-dev"
+			"dev-trunk": "3.6.x-dev"
 		},
 		"version-constants": {
 			"::PACKAGE_VERSION": "src/class-initializer.php"

diff --git a/projects/packages/my-jetpack/package.json b/projects/packages/my-jetpack/package.json
@@ -1,7 +1,7 @@
 {
 	"private": true,
 	"name": "@automattic/jetpack-my-jetpack",
-	"version": "3.5.0",
+	"version": "3.6.0-alpha",
 	"description": "WP Admin page with information and configuration shared among all Jetpack stand-alone plugins",
 	"homepage": "https://github.com/Automattic/jetpack/tree/HEAD/projects/packages/my-jetpack/#readme",
 	"bugs": {

diff --git a/projects/packages/my-jetpack/src/class-initializer.php b/projects/packages/my-jetpack/src/class-initializer.php
@@ -31,7 +31,7 @@ class Initializer {
 	 *
 	 * @var string
 	 */
-	const PACKAGE_VERSION = '3.5.0';
+	const PACKAGE_VERSION = '3.6.0-alpha';
 
 	/**
 	 * HTML container ID for the IDC screen on My Jetpack page.
@@ -231,6 +231,7 @@ public static function register_rest_endpoints() {
 		new REST_Products();
 		new REST_Purchases();
 		new REST_Zendesk_Chat();
+		new REST_Product_Data();
 		new REST_AI();
 
 		register_rest_route(

diff --git a/projects/packages/my-jetpack/src/class-rest-product-data.php b/projects/packages/my-jetpack/src/class-rest-product-data.php
@@ -0,0 +1,58 @@
+<?php
+/**
+ * Sets up the Product Data REST API endpoints.
+ *
+ * @package automattic/my-jetpack
+ */
+
+namespace Automattic\Jetpack\My_Jetpack;
+
+use Automattic\Jetpack\Connection\Client;
+use WP_Error;
+
+/**
+ * Registers the REST routes for Product Data
+ */
+class REST_Product_Data {
+	/**
+	 * Constructor.
+	 */
+	public function __construct() {
+		register_rest_route(
+			'my-jetpack/v1',
+			'site/product-data',
+			array(
+				'methods'             => \WP_REST_Server::READABLE,
+				'callback'            => __CLASS__ . '::get_all_product_data',
+				'permission_callback' => __CLASS__ . '::permissions_callback',
+			)
+		);
+	}
+
+	/**
+	 * Checks if the user has the correct permissions
+	 */
+	public static function permissions_callback() {
+		return current_user_can( 'manage_options' );
+	}
+
+	/**
+	 * Gets the product data for all products
+	 *
+	 * @return array|WP_Error
+	 */
+	public static function get_all_product_data() {
+		$site_id        = \Jetpack_Options::get_option( 'id' );
+		$wpcom_endpoint = sprintf( 'sites/%d/jetpack-product-data?locale=%2$s&force=wpcom', $site_id, get_user_locale() );
+		$api_version    = '2';
+		$response       = Client::wpcom_json_api_request_as_blog( $wpcom_endpoint, $api_version, array(), null, 'wpcom' );
+		$response_code  = wp_remote_retrieve_response_code( $response );
+		$body           = json_decode( wp_remote_retrieve_body( $response ) );
+
+		if ( is_wp_error( $response ) || empty( $response['body'] ) || 200 !== $response_code ) {
+			return new WP_Error( 'site_products_data_fetch_failed', 'Site products data fetch failed', array( 'status' => $response_code ? $response_code : 400 ) );
+		}
+
+		return rest_ensure_response( $body, 200 );
+	}
+}
diff --git a/projects/packages/videopress/changelog/add-bi-yearly-constants-to-videopress-plugin b/projects/packages/videopress/changelog/add-bi-yearly-constants-to-videopress-plugin
@@ -0,0 +1,4 @@
+Significance: patch
+Type: fixed
+
+Add bi-yearly constants for complete and videopress in config
diff --git a/projects/packages/videopress/package.json b/projects/packages/videopress/package.json
@@ -1,7 +1,7 @@
 {
 	"private": true,
 	"name": "@automattic/jetpack-videopress",
-	"version": "0.17.0",
+	"version": "0.17.1-alpha",
 	"description": "VideoPress package",
 	"homepage": "https://github.com/Automattic/jetpack/tree/HEAD/projects/packages/videopress/#readme",
 	"bugs": {

diff --git a/projects/packages/videopress/src/class-package-version.php b/projects/packages/videopress/src/class-package-version.php
@@ -11,7 +11,7 @@
  * The Package_Version class.
  */
 class Package_Version {
-	const PACKAGE_VERSION = '0.17.0';
+	const PACKAGE_VERSION = '0.17.1-alpha';
 
 	const PACKAGE_SLUG = 'videopress';
 

diff --git a/projects/plugins/backup/changelog/add-jetpack-endpoint-for-jetpack-product-data b/projects/plugins/backup/changelog/add-jetpack-endpoint-for-jetpack-product-data
@@ -0,0 +1,4 @@
+Significance: patch
+Type: added
+
+
diff --git a/projects/plugins/backup/composer.lock b/projects/plugins/backup/composer.lock
diff --git a/projects/plugins/boost/changelog/add-jetpack-endpoint-for-jetpack-product-data b/projects/plugins/boost/changelog/add-jetpack-endpoint-for-jetpack-product-data
@@ -0,0 +1,4 @@
+Significance: patch
+Type: added
+
+
diff --git a/projects/plugins/boost/composer.lock b/projects/plugins/boost/composer.lock
diff --git a/projects/plugins/jetpack/changelog/add-jetpack-endpoint-for-jetpack-product-data b/projects/plugins/jetpack/changelog/add-jetpack-endpoint-for-jetpack-product-data
@@ -0,0 +1,4 @@
+Significance: patch
+Type: other
+
+