Skip to content

Commit e3f642d

Browse files
committed
fix(project): adjust minor things
1 parent d9cfedf commit e3f642d

File tree

1 file changed

+21
-29
lines changed

1 file changed

+21
-29
lines changed

src/index.tsx

Lines changed: 21 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -1,37 +1,28 @@
11
import * as React from 'react';
22
import { useAppendHeadScript } from './useAppendHeadScript';
33

4-
type TInitHotjar = {
5-
hotjarId: string;
6-
hotjarVersion: string;
7-
shouldLog?: boolean;
8-
};
9-
10-
type TUserInfo = {
11-
userInfo: {
12-
id: string;
13-
};
14-
shouldLog?: boolean;
15-
};
16-
174
type TUseHotjar = {
18-
initHotjar: ({
19-
hotjarId,
20-
hotjarVersion,
21-
shouldLog,
22-
}: TInitHotjar) => Promise<boolean>;
23-
identityHotjar: ({ userInfo }: TUserInfo) => Promise<boolean>;
5+
initHotjar: (
6+
hotjarId: string,
7+
hotjarVersion: string,
8+
shouldLog?: boolean
9+
) => Promise<boolean>;
10+
identityHotjar: (
11+
userId: string,
12+
userInfo: string,
13+
shouldLog?: boolean
14+
) => Promise<boolean>;
2415
};
2516

2617
export function useHotjar(): TUseHotjar {
2718
const { appendHeadScript } = useAppendHeadScript();
2819

2920
const initHotjar = React.useCallback(
30-
async ({
31-
hotjarId,
32-
hotjarVersion,
33-
shouldLog = true,
34-
}: TInitHotjar): Promise<boolean> => {
21+
async (
22+
hotjarId: string,
23+
hotjarVersion: string,
24+
shouldLog = true
25+
): Promise<boolean> => {
3526
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=');`;
3627
const isHotjarAppended = await appendHeadScript({
3728
scriptText: hotjarScript,
@@ -46,12 +37,13 @@ export function useHotjar(): TUseHotjar {
4637
);
4738

4839
const identityHotjar = React.useCallback(
49-
async ({ userInfo, shouldLog = true }: TUserInfo): Promise<boolean> => {
40+
async (
41+
userId: string,
42+
userInfo: string,
43+
shouldLog = true
44+
): Promise<boolean> => {
5045
try {
51-
const { id, ...restUserInfo } = userInfo;
52-
const hotjarIdentityScript = `var userId="${id}" || null;window.hj("identify",userId,{${JSON.stringify(
53-
restUserInfo
54-
)}});`;
46+
const hotjarIdentityScript = `var userId="${userId}" || null;window.hj("identify",userId,{${userInfo}}});`;
5547
const isIdentified = await appendHeadScript({
5648
scriptText: hotjarIdentityScript,
5749
scriptId: 'identity-script',

0 commit comments

Comments
 (0)