Merge pull request #4263 from Shopify/include-api-version-in-complian…

…ce-webhooks

Include the webhook api version in the privacy compliance webhooks

packages/app/src/cli/models/extensions/specifications/app_config_privacy_compliance_webhooks.test.ts

@@ -34,6 +34,7 @@ describe('privacy_compliance_webhooks', () => {
       // Given
       const object = {
         webhooks: {
+          api_version: '2024-07',
           subscriptions: [
             {
               compliance_topics: ['customers/redact', 'customers/data_request'],
@@ -53,6 +54,7 @@ describe('privacy_compliance_webhooks', () => {
 
       // Then
       expect(result).toMatchObject({
+        api_version: '2024-07',
         customers_redact_url: 'https://example.com/customers_webhooks',
         customers_data_request_url: 'https://example.com/customers_webhooks',
         shop_redact_url: 'https://example.com/shop_webhooks',

packages/app/src/cli/models/extensions/specifications/app_config_privacy_compliance_webhooks.ts

@@ -31,11 +31,22 @@ function transformToPrivacyComplianceWebhooksModule(content: object, appConfigur
     appUrl = (appConfiguration as CurrentAppConfiguration)?.application_url
   }
 
-  return compact({
+  const urls = compact({
     customers_redact_url: relativeUri(getCustomersDeletionUri(webhooks), appUrl),
     customers_data_request_url: relativeUri(getCustomersDataRequestUri(webhooks), appUrl),
     shop_redact_url: relativeUri(getShopDeletionUri(webhooks), appUrl),
   })
+
+  if (Object.keys(urls).length === 0) {
+    return urls
+  } else {
+    // eslint-disable-next-line @typescript-eslint/naming-convention
+    const {api_version} = webhooks
+    return {
+      api_version,
+      ...urls,
+    }
+  }
 }
 
 function transformFromPrivacyComplianceWebhooksModule(content: object, options?: {flags?: Flag[]}) {