Merge pull request #3322 from withspectrum/remove-console-warn-amplitude

Remove amplitude console warns

Author: brianlovin

shared/clients/analytics/raven.js

@@ -9,6 +9,4 @@ if (
     whitelistUrls: [/spectrum\.chat/, /www\.spectrum\.chat/],
     environment: process.env.NODE_ENV,
   }).install();
-} else {
-  console.warn('Raven not enabled locally');
 }

shared/clients/analytics/setUser.js

@@ -3,7 +3,6 @@ import type { Amplitude } from './';
 
 export const createSetUser = (amplitude: Amplitude) => (userId: string) => {
   if (!amplitude) {
-    console.warn('No amplitude function attached to window');
     return;
   }
 
@@ -13,12 +12,10 @@ export const createSetUser = (amplitude: Amplitude) => (userId: string) => {
       : process.env.AMPLITUDE_API_KEY_DEVELOPMENT;
 
   if (!AMPLITUDE_API_KEY) {
-    // console.warn(`[Amplitude Dev] Set user ${userId}`);
     return;
   }
 
   const amplitudePromise = () => {
-    // console.warn(`[Amplitude] Set user ${userId}`);
     return amplitude.getInstance().setUserId(userId);
   };
 

shared/clients/analytics/track.js

@@ -6,7 +6,6 @@ export const createTrack = (amplitude: Amplitude, client: AmplitudeClient) => (
   eventProperties?: Object = {}
 ) => {
   if (!amplitude) {
-    console.warn('No amplitude function attached to window');
     return;
   }
 
@@ -16,12 +15,10 @@ export const createTrack = (amplitude: Amplitude, client: AmplitudeClient) => (
       : process.env.AMPLITUDE_API_KEY_DEVELOPMENT;
 
   if (!AMPLITUDE_API_KEY) {
-    // console.warn(`[Amplitude Dev] Tracking ${eventType}`);
     return;
   }
 
   const amplitudePromise = () => {
-    // console.warn(`[Amplitude] Tracking ${eventType}`);
     return amplitude.getInstance().logEvent(eventType, {
       ...eventProperties,
       client,

shared/clients/analytics/unsetUser.js

@@ -3,7 +3,6 @@ import type { Amplitude } from './';
 
 export const createUnsetUser = (amplitude: Amplitude) => () => {
   if (!amplitude) {
-    console.warn('No amplitude function attached to window');
     return;
   }
 
@@ -13,12 +12,10 @@ export const createUnsetUser = (amplitude: Amplitude) => () => {
       : process.env.AMPLITUDE_API_KEY_DEVELOPMENT;
 
   if (!AMPLITUDE_API_KEY) {
-    // console.warn(`[Amplitude Dev] Unset user`);
     return;
   }
 
   const amplitudePromise = () => {
-    // console.warn('[Amplitude] Unset user');
     return amplitude.getInstance().setUserId(null);
   };