Replace inline logging with logToConsole plugin

prezly · Jan 9, 2025 · 9438f61 · 9438f61
1 parent c32181d
commit 9438f61
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 21 deletions.
diff --git a/packages/analytics-nextjs/src/AnalyticsProvider.tsx b/packages/analytics-nextjs/src/AnalyticsProvider.tsx
@@ -10,7 +10,7 @@ import type { PropsWithChildren } from 'react';
 import { createContext, useContext, useEffect, useState } from 'react';
 
 import { isTrackingCookieAllowed } from './lib';
-import { normalizePrezlyMetaPlugin, sendEventToPrezlyPlugin } from './plugins';
+import { logToConsole, normalizePrezlyMetaPlugin, sendEventToPrezlyPlugin } from './plugins';
 import { TrackingPolicy } from './types';
 import type {
     PickedGalleryProperties,
@@ -99,6 +99,7 @@ export function AnalyticsProvider({
                             ...(uuid
                                 ? [sendEventToPrezlyPlugin(uuid), normalizePrezlyMetaPlugin()]
                                 : []),
+                            ...(process.env.NODE_ENV === 'production' ? [] : [logToConsole()]),
                             ...(plugins || []),
                         ],
                     },

diff --git a/packages/analytics-nextjs/src/hooks/useAnalytics.ts b/packages/analytics-nextjs/src/hooks/useAnalytics.ts
@@ -67,11 +67,6 @@ export function useAnalytics() {
         (userId: string, traits: object = {}, callback?: () => void) => {
             const extendedTraits = injectPrezlyMeta(traits);
 
-            if (process.env.NODE_ENV !== 'production') {
-                // eslint-disable-next-line no-console
-                console.log(`analytics.identify(${stringify(userId, extendedTraits)})`);
-            }
-
             if (trackingPolicy === TrackingPolicy.CONSENT_TO_IDENTIFY && !consent) {
                 setDeferredIdentity({ userId, traits: extendedTraits });
                 if (callback) {
@@ -92,11 +87,6 @@ export function useAnalytics() {
 
     const alias = useCallback(
         (userId: string, previousId: string) => {
-            if (process.env.NODE_ENV !== 'production') {
-                // eslint-disable-next-line no-console
-                console.log(`analytics.alias(${stringify(userId, previousId)})`);
-            }
-
             addToQueue(() => {
                 if (analyticsRef.current && analyticsRef.current.alias) {
                     analyticsRef.current.alias(userId, previousId);
@@ -110,11 +100,6 @@ export function useAnalytics() {
         (category?: string, name?: string, properties: object = {}, callback?: () => void) => {
             const extendedProperties = injectPrezlyMeta(properties);
 
-            if (process.env.NODE_ENV !== 'production') {
-                // eslint-disable-next-line no-console
-                console.log(`analytics.page(${stringify(category, name, extendedProperties)})`);
-            }
-
             addToQueue(() => {
                 if (analyticsRef.current && analyticsRef.current.page) {
                     analyticsRef.current.page(category, name, extendedProperties, {}, callback);
@@ -128,11 +113,6 @@ export function useAnalytics() {
         (event: string, properties: object = {}, callback?: () => void) => {
             const extendedProperties = injectPrezlyMeta(properties);
 
-            if (process.env.NODE_ENV !== 'production') {
-                // eslint-disable-next-line no-console
-                console.log(`analytics.track(${stringify(event, extendedProperties)})`);
-            }
-
             addToQueue(() => {
                 if (analyticsRef.current && analyticsRef.current.track) {
                     analyticsRef.current.track(event, extendedProperties, {}, callback);