Skip to content

fix: posthog events trigger in staging - #3220

Merged
sriramveeraghanta merged 3 commits into
developfrom
fix/posthog
Dec 22, 2023
Merged

fix: posthog events trigger in staging#3220
sriramveeraghanta merged 3 commits into
developfrom
fix/posthog

Conversation

@rameshkumarchandra

Copy link
Copy Markdown
Member

This PR contains the fix for PostHog events which are not getting triggered in staging and production.

Comment thread web/lib/wrappers/posthog-wrapper.tsx Outdated
@@ -48,7 +48,7 @@ const PosthogWrapper: FC<IPosthogWrapper> = (props) => {
});
}
// eslint-disable-next-line react-hooks/exhaustive-deps

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

delete this comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix any issue with exhaustive-deps

Comment thread web/lib/wrappers/posthog-wrapper.tsx Outdated
};
// eslint-disable-next-line react-hooks/exhaustive-deps
}, []);
}, [router.events]);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is not required here.

@sriramveeraghanta
sriramveeraghanta merged commit fe95193 into develop Dec 22, 2023
@sriramveeraghanta
sriramveeraghanta deleted the fix/posthog branch December 22, 2023 08:35
sriramveeraghanta added a commit that referenced this pull request Dec 22, 2023
fix: posthog events trigger in staging (#3220)
JayashTripathy pushed a commit that referenced this pull request May 22, 2025
* refactor: work item widget code split

* fix: build errors
DoctorFogarty pushed a commit to DoctorFogarty/plane that referenced this pull request Aug 7, 2026
* fix: posthog events trigger in staging

* refactor: cleared comment lines for warnings, added dependancy values

* refactor: removed unnecessary dependency
DoctorFogarty pushed a commit to DoctorFogarty/plane that referenced this pull request Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants