zaid.marzguioui/myeasycms-v2
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Refactor analytics event handling and manager setup

Browse Source 
Updated event handling in `analytics-provider.tsx` to return results, enhancing consistency. Added a new function `registerActiveServices` in `analytics-manager.ts` for initializing active services, streamlining provider setup. Clarified types for `AnalyticsProviderFactory` to accept optional configuration.
This commit is contained in:
gbuomprisco
2024-07-26 21:09:02 +02:00
parent b9e9b8af48
commit 4eef94fcd6
3 changed files with 28 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
apps/web/components/analytics-provider.tsx
View File

@@ -15,7 +15,7 @@ import {
type AnalyticsMapping<
T extends ConsumerProvidedEventTypes = NonNullable<unknown>,
> = {
[K in AppEventType<T>]?: (event: AppEvent<T, K>) => void;
[K in AppEventType<T>]?: (event: AppEvent<T, K>) => unknown;
};
/**
@@ -51,17 +51,17 @@ const analyticsMapping: AnalyticsMapping = {
const userId = event.payload.userId;
if (userId) {
analytics.identify(userId);
return analytics.identify(userId);
}
},
'user.signedUp': (event) => {
analytics.trackEvent(event.type, event.payload);
return analytics.trackEvent(event.type, event.payload);
},
'checkout.started': (event) => {
analytics.trackEvent(event.type, event.payload);
return analytics.trackEvent(event.type, event.payload);
},
'user.updated': (event) => {
analytics.trackEvent(event.type, event.payload);
return analytics.trackEvent(event.type, event.payload);
},
};
@@ -83,7 +83,7 @@ export function AnalyticsProvider(props: React.PropsWithChildren) {
* Hook to report page views to the analytics service
* @param reportAnalyticsFn
*/
function useReportPageView(reportAnalyticsFn: (url: string) => void) {
function useReportPageView(reportAnalyticsFn: (url: string) => unknown) {
const pathname = usePathname();
const searchParams = useSearchParams();