Skip to content

Commit 7ebcd18

Browse files
committed
fix/posthog-event
1 parent 480b8df commit 7ebcd18

File tree

4 files changed

+9
-10
lines changed

4 files changed

+9
-10
lines changed

src/Shared/Components/GettingStartedCard/GettingStarted.tsx

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -24,20 +24,20 @@ import './gettingStarted.scss'
2424
import { Button, ButtonStyleType, ButtonVariantType } from '../Button'
2525

2626
const GettingStartedCard = ({ className, hideGettingStartedCard }: GettingStartedType) => {
27-
const onClickedOkay = async (e) => {
27+
const onClickedOkay = async () => {
2828
setActionWithExpiry('clickedOkay', 1)
2929
hideGettingStartedCard()
30-
await handlePostHogEventUpdate(e)
30+
await handlePostHogEventUpdate(POSTHOG_EVENT_ONBOARDING.TOOLTIP_OKAY)
3131
}
3232

33-
const onClickedDontShowAgain = async (e) => {
33+
const onClickedDontShowAgain = async () => {
3434
const updatedPayload = {
3535
key: LOGIN_COUNT,
3636
value: `${MAX_LOGIN_COUNT}`,
3737
}
3838
await updateLoginCount(updatedPayload)
3939
hideGettingStartedCard(updatedPayload.value)
40-
await handlePostHogEventUpdate(e)
40+
await handlePostHogEventUpdate(POSTHOG_EVENT_ONBOARDING.TOOLTIP_DONT_SHOW_AGAIN)
4141
}
4242

4343
return (
@@ -53,15 +53,13 @@ const GettingStartedCard = ({ className, hideGettingStartedCard }: GettingStarte
5353
size={ComponentSizeType.xs}
5454
dataTestId="getting-started-okay"
5555
onClick={onClickedOkay}
56-
data-posthog={POSTHOG_EVENT_ONBOARDING.TOOLTIP_OKAY}
5756
/>
5857
<Button
5958
text="Don't show again"
6059
size={ComponentSizeType.xs}
6160
dataTestId="getting-started-don't-show-again"
6261
onClick={onClickedDontShowAgain}
6362
style={ButtonStyleType.neutral}
64-
data-posthog={POSTHOG_EVENT_ONBOARDING.TOOLTIP_DONT_SHOW_AGAIN}
6563
variant={ButtonVariantType.secondary}
6664
/>
6765
</div>

src/Shared/Components/Header/PageHeader.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ const PageHeader = ({
100100
hideGettingStartedCard()
101101
}
102102

103-
const onClickHelp = async (e) => {
103+
const onClickHelp = async () => {
104104
if (
105105
!window._env_.K8S_CLIENT &&
106106
currentServerInfo.serverInfo?.installationType !== InstallationType.ENTERPRISE
@@ -113,7 +113,7 @@ const PageHeader = ({
113113
}
114114
setActionWithExpiry('clickedOkay', 1)
115115
hideGettingStartedCard()
116-
await handlePostHogEventUpdate(e, POSTHOG_EVENT_ONBOARDING.HELP)
116+
await handlePostHogEventUpdate(POSTHOG_EVENT_ONBOARDING.HELP)
117117
ReactGA.event({
118118
category: 'Main Navigation',
119119
action: `Help Clicked`,

src/Shared/Components/Header/index.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,3 +16,4 @@
1616

1717
export { default as PageHeader } from './PageHeader'
1818
export * from './HeaderWithCreateButton'
19+
export * from './utils'

src/Shared/Components/Header/utils.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,9 @@ import { LOGIN_COUNT } from '../../../Common'
2020
const millisecondsInDay = 86400000
2121
export const getDateInMilliseconds = (days) => 1 + new Date().valueOf() + (days ?? 0) * millisecondsInDay
2222

23-
export const handlePostHogEventUpdate = async (e, eventName?: string): Promise<void> => {
23+
export const handlePostHogEventUpdate = async (eventName?: string): Promise<void> => {
2424
const payload = {
25-
eventType: eventName || e.target?.dataset.posthog,
25+
eventType: eventName,
2626
key: LOGIN_COUNT,
2727
value: '',
2828
active: true,

0 commit comments

Comments
 (0)