Remove flow param for Events API requests

newfold-labs · Jun 24, 2024 · 87089f8 · 87089f8
1 parent 243277e
commit 87089f8
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/.github/workflows/lint-check-php.yml b/.github/workflows/lint-check-php.yml
@@ -8,6 +8,7 @@ on:
     types: [opened, edited, reopened, ready_for_review]
     paths:
       - '**.php'
+      - '!build/**/*.php'
   workflow_dispatch:
 
 concurrency:

diff --git a/src/OnboardingSPA/utils/api/common.js b/src/OnboardingSPA/utils/api/common.js
@@ -5,10 +5,10 @@ import {
 	migrateRestBase,
 } from '../../../constants';
 
-export const onboardingRestURL = ( api ) => {
+export const onboardingRestURL = ( api, includeFlow = true ) => {
 	return (
 		`${ onboardingRestBase }/${ api }` +
-		( window.nfdOnboarding?.currentFlow
+		( includeFlow && window.nfdOnboarding?.currentFlow
 			? `&flow=${ window.nfdOnboarding.currentFlow }`
 			: '' )
 	);

diff --git a/src/onboarding.js b/src/onboarding.js
@@ -14,8 +14,8 @@ if ( runtimeDataExists ) {
 		HiiveAnalytics.initialize( {
 			namespace: CATEGORY,
 			urls: {
-				single: onboardingRestURL( 'events' ),
-				batch: onboardingRestURL( 'events/batch' ),
+				single: onboardingRestURL( 'events', false ),
+				batch: onboardingRestURL( 'events/batch', false ),
 			},
 			settings: {
 				debounce: {