yujunjun/OpenMetadata
1
0
Fork 0
mirror of https://github.com/open-metadata/OpenMetadata.git synced 2025-12-28 07:58:31 +00:00
Code Issues Packages Projects Releases Wiki Activity

Minor: invoke analytics instance only when it's available (#14636)

Browse Source
This commit is contained in:
Sachin Chaurasiya 2024-01-09 13:35:40 +05:30 committed by GitHub
parent 2832bbcc8c
commit e08891fbad
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
openmetadata-ui/src/main/resources/ui/src/components/AppRouter/AppRouter.tsx
View File

@ -11,6 +11,7 @@
* limitations under the License.
*/
import { isNil } from 'lodash';
import React, { useCallback, useEffect } from 'react';
import { Redirect, Route, Switch, useLocation } from 'react-router-dom';
import { useAnalytics } from 'use-analytics';
@ -80,8 +81,9 @@ const AppRouter = () => {
/**
* Ignore the slash path because we are treating my data as
* default path.
* And check if analytics instance is available
*/
if (pathname !== '/') {
if (pathname !== '/' && !isNil(analytics)) {
// track page view on route change
analytics.page();
}
@ -91,7 +93,11 @@ const AppRouter = () => {
(event: MouseEvent) => {
const eventValue =
(event.target as HTMLElement)?.textContent || CustomEventTypes.Click;
if (eventValue) {
/**
* Ignore the click event if the event value is undefined
* And analytics instance is not available
*/
if (eventValue && !isNil(analytics)) {
analytics.track(eventValue);
}
},