Making integration_data data not mandatory.

[message-dimke]

Changes proposed in this Pull Request:

Integration data appeared not mandatory for the extension to function. However, we were using it to identify if the user connected status. Since the Pinterest Commerce Integration request may fail, we do not want the extension to rely on its presence.

Detailed test instructions:

1. Add a filter to make the APIV5::create_commerce_integration call fail.

add_filter(
	'pre_http_request',
	function ( $response, $parsed_args, $url ) {
		if (
			'https://api.pinterest.com/v5/integrations/commerce' === $url &&
			'POST' === $parsed_args['method']
		) {
			return array(
				'headers' => array(
					'content-type' => 'application/json',
				),
				'body' => json_encode(
					array(
						'code'    => 911,
						'message' => 'Oops! Something went wrong!',
					)
				),
				'response' => array(
					'code'    => 500,
					'message' => 'Unexpected error',
				),
				'cookies'  => array(),
				'filename' => '',
			);
		}

		return $response;
	},
	10,
	3
);

pinterest-for-woocommerce/src/API/APIV5.php

Lines 407 to 413 in f48b082

	public static function create_commerce_integration( array $integration_data ): array {
	return self::make_request(
	'integrations/commerce',
	'POST',
	$integration_data
	);
	}

2. If you are connected to Pinterest - disconnect. Otherwise, connect to your Pinterest account using the extension.
3. You should not see any changes. Nothing must fail.

Changelog entry

Update - Make integration_data optional for the extension.

[budzanowski]

Looks good!