chore(project): refactor things

Author: olavoparno

package-lock.json

@@ -31,19 +31,16 @@
     ]
   },
   "scripts": {
-    "test": "cross-env CI=1 react-scripts test --env=jsdom || true",
+    "test": "CI=true react-scripts test --env=jsdom || true",
     "test:watch": "react-scripts test --env=jsdom",
     "build": "rollup -c",
     "start": "rollup -c -w",
     "prepare": "npm run build",
-    "predeploy": "cd example && npm install && npm run build",
-    "deploy": "gh-pages -d example/build",
     "format": "prettier --write src/**/*.{js,jsx,ts,tsx}",
     "lint": "eslint --fix src/**/*.{js,jsx,ts,tsx}",
     "make-badges": "echo ok",
     "release": "standard-version"
   },
-  "dependencies": {},
   "peerDependencies": {
     "react": "^16.13.1"
   },
@@ -55,7 +52,6 @@
     "@types/react": "^16.9.46",
     "@typescript-eslint/eslint-plugin": "^3.9.0",
     "@typescript-eslint/parser": "^3.9.0",
-    "cross-env": "^7.0.2",
     "eslint": "^7.6.0",
     "eslint-config-airbnb": "^18.2.0",
     "eslint-config-airbnb-typescript-prettier": "^3.1.0",
@@ -65,7 +61,6 @@
     "eslint-plugin-prettier": "^3.1.4",
     "eslint-plugin-react": "^7.20.5",
     "eslint-plugin-react-hooks": "^4.0.8",
-    "gh-pages": "^3.1.0",
     "husky": "^4.2.5",
     "prettier": "^2.0.5",
     "react": "^16.13.1",
@@ -76,7 +71,6 @@
     "rollup-plugin-node-resolve": "^5.2.0",
     "rollup-plugin-peer-deps-external": "^2.2.3",
     "rollup-plugin-typescript2": "^0.27.2",
-    "rollup-plugin-url": "^3.0.1",
     "standard-version": "^8.0.2",
     "typescript": "^3.9.7"
   }

package.json

@@ -1,47 +1,46 @@
 import * as React from 'react';
-import { useAppendHeadScript } from './useAppendHeadScript';
+import useAppendHeadScript from './useAppendHeadScript';
 
 type TUseHotjar = {
   initHotjar: (
     hotjarId: string,
     hotjarVersion: string,
-    shouldLog?: boolean
+    logCallback?: () => void
   ) => boolean;
   identityHotjar: (
     userId: string,
     userInfo: string,
-    shouldLog?: boolean
+    logCallback?: () => void
   ) => boolean;
 };
 
 export function useHotjar(): TUseHotjar {
   const { appendHeadScript } = useAppendHeadScript();
 
   const initHotjar = React.useCallback(
-    (hotjarId: string, hotjarVersion: string, shouldLog = true): boolean => {
+    (hotjarId: string, hotjarVersion: string, loggerFunction): boolean => {
       const hotjarScript = `(function(h,o,t,j,a,r){h.hj=h.hj||function(){(h.hj.q=h.hj.q||[]).push(arguments)};h._hjSettings={hjid:${hotjarId},hjsv:${hotjarVersion}};a=o.getElementsByTagName('head')[0];r=o.createElement('script');r.async=1;r.src=t+h._hjSettings.hjid+j+h._hjSettings.hjsv;a.appendChild(r);})(window,document,'https://static.hotjar.com/c/hotjar-','.js?sv=');`;
-      const isHotjarAppended = appendHeadScript({
-        scriptText: hotjarScript,
-        scriptId: 'hotjar-script',
-      });
+      const isHotjarAppended = appendHeadScript(hotjarScript, 'hotjar-script');
 
-      if (shouldLog) console.info(`Hotjar ready: ${isHotjarAppended}`);
+      if (loggerFunction && typeof loggerFunction === 'function')
+        loggerFunction(`Hotjar ready: ${isHotjarAppended}`);
 
       return isHotjarAppended;
     },
     [appendHeadScript]
   );
 
   const identityHotjar = React.useCallback(
-    (userId: string, userInfo: string, shouldLog = true): boolean => {
+    (userId: string, userInfo: string, loggerFunction): boolean => {
       try {
         const hotjarIdentityScript = `var userId="${userId}" || null;window.hj("identify",userId,${userInfo});`;
-        const isIdentified = appendHeadScript({
-          scriptText: hotjarIdentityScript,
-          scriptId: 'identity-script',
-        });
+        const isIdentified = appendHeadScript(
+          hotjarIdentityScript,
+          'identity-script'
+        );
 
-        if (shouldLog) console.info(`Hotjar identified: ${isIdentified}`);
+        if (loggerFunction && typeof loggerFunction === 'function')
+          loggerFunction(`Hotjar identified: ${isIdentified}`);
 
         return isIdentified;
       } catch (error) {
@@ -54,7 +53,7 @@ export function useHotjar(): TUseHotjar {
   );
 
   return React.useMemo(() => ({ initHotjar, identityHotjar }), [
-    identityHotjar,
     initHotjar,
+    identityHotjar,
   ]);
 }