Update/all tracks to snake case (#38576)

* Update all tracks to snake case and improve typing * changelog
Automattic · Jul 29, 2024 · 5883fd3 · 5883fd3
1 parent 5aef951
commit 5883fd3
Show file tree

Hide file tree

Showing 6 changed files with 13 additions and 9 deletions.
diff --git a/projects/packages/my-jetpack/_inc/components/connection-status-card/index.tsx b/projects/packages/my-jetpack/_inc/components/connection-status-card/index.tsx
@@ -196,8 +196,8 @@ const ConnectionStatusCard: ConnectionStatusCardType = ( {
 	const hasSiteConnectionBrokenModules = brokenModules?.needs_site_connection.length > 0;
 	const tracksEventData = useMemo( () => {
 		return {
-			userConnectionBrokenModules: brokenModules?.needs_user_connection.join( ', ' ),
-			siteConnectionBrokenModules: brokenModules?.needs_site_connection.join( ', ' ),
+			user_connection_broken_modules: brokenModules?.needs_user_connection.join( ', ' ),
+			site_connection_broken_modules: brokenModules?.needs_site_connection.join( ', ' ),
 		};
 	}, [ brokenModules ] );
 
@@ -209,7 +209,7 @@ const ConnectionStatusCard: ConnectionStatusCardType = ( {
 			e && e.preventDefault();
 			recordEvent( 'jetpack_myjetpack_connection_manage_dialog_click', {
 				...tracksEventData,
-				connectionType,
+				connection_type: connectionType,
 			} );
 			setIsManageConnectionDialogOpen( true );
 		},

diff --git a/projects/packages/my-jetpack/_inc/components/my-jetpack-screen/index.jsx b/projects/packages/my-jetpack/_inc/components/my-jetpack-screen/index.jsx
@@ -44,7 +44,7 @@ const GlobalNotice = ( { message, title, options } ) => {
 		const tracksArgs = options?.tracksArgs || {};
 
 		recordEvent( 'jetpack_myjetpack_global_notice_view', {
-			noticeId: options.id,
+			notice_id: options.id,
 			...tracksArgs,
 		} );
 	}, [ options.id, recordEvent, options?.tracksArgs ] );

diff --git a/...ges/my-jetpack/_inc/components/product-cards-section/protect-card/scan-threats-status.tsx b/...ges/my-jetpack/_inc/components/product-cards-section/protect-card/scan-threats-status.tsx
@@ -208,7 +208,7 @@ function ScanStatus( { status }: { status: 'success' | 'partial' | 'off' } ) {
 						tracksEventProps={ {
 							location: 'scan',
 							status: status,
-							hasPaidPlan: false,
+							has_paid_plan: false,
 							threats: 0,
 						} }
 					>

diff --git a/projects/packages/my-jetpack/_inc/hooks/use-analytics/index.ts b/projects/packages/my-jetpack/_inc/hooks/use-analytics/index.ts
@@ -5,7 +5,7 @@ import useMyJetpackConnection from '../use-my-jetpack-connection';
 
 type TracksRecordEvent = (
 	event: `jetpack_${ string }`, // Enforces the event name to start with "jetpack_"
-	properties?: Record< string, unknown >
+	properties?: Record< Lowercase< string >, unknown >
 ) => void;
 
 const useAnalytics = () => {
@@ -44,8 +44,8 @@ const useAnalytics = () => {
 		jetpackAnalytics.tracks.recordEvent( event, {
 			...properties,
 			version: myJetpackVersion,
-			isSiteConnected,
-			isUserConnected,
+			is_site_connected: isSiteConnected,
+			is_user_connected: isUserConnected,
 			referring_plugins: connectedPluginsSlugs,
 		} );
 		// eslint-disable-next-line react-hooks/exhaustive-deps

diff --git a/...ts/packages/my-jetpack/_inc/hooks/use-notification-watcher/use-site-connection-notice.tsx b/...ts/packages/my-jetpack/_inc/hooks/use-notification-watcher/use-site-connection-notice.tsx
@@ -113,7 +113,7 @@ const useSiteConnectionNotice = ( redBubbleAlerts: RedBubbleAlerts ) => {
 			isRedBubble: true,
 			tracksArgs: {
 				type: connectionError.type,
-				isError: connectionError.is_error,
+				is_error: connectionError.is_error,
 			},
 		};
 

diff --git a/projects/packages/my-jetpack/changelog/update-all-tracks-to-snake-case b/projects/packages/my-jetpack/changelog/update-all-tracks-to-snake-case
@@ -0,0 +1,4 @@
+Significance: patch
+Type: fixed
+
+Update all tracks to snake case, camel case is not supported