Skip to content

Commit 7337613

Browse files
committed
refactor(analytics): Remove click tracking from events and hackathons
- Remove trackClick function calls from event and hackathon registration handlers - Eliminate click tracking from useAnalyticsTracking hook initialization - Remove clicks field from AnalyticsCharts component state and calculations - Remove MousePointerClick icon import from lucide-react - Simplify analytics data transformation by removing click proportion distribution logic - Remove clicks-related remainder calculations and distribution logic - Update chart data mapping to exclude clicksRemainder field - Remove clicks from analytics totals calculation in CompanyDashboard - Simplify next.config.ts by removing unnecessary configuration - Delete obsolete delete-test-company.js cleanup script - Streamline analytics tracking to focus on views and registrations only
1 parent 7545be8 commit 7337613

File tree

8 files changed

+24
-454
lines changed

8 files changed

+24
-454
lines changed

app/events/[slug]/page.tsx

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -98,11 +98,11 @@ export default function EventPage() {
9898

9999
// Track analytics
100100
console.log('[EventPage] About to call useAnalyticsTracking with slug:', slug)
101-
const { trackClick } = useAnalyticsTracking({
101+
useAnalyticsTracking({
102102
eventSlug: slug,
103103
trackView: true,
104104
})
105-
console.log('[EventPage] useAnalyticsTracking returned:', { trackClick })
105+
console.log('[EventPage] useAnalyticsTracking initialized')
106106

107107
useEffect(() => {
108108
const fetchEvent = async () => {
@@ -143,9 +143,6 @@ export default function EventPage() {
143143
const handleRegister = async () => {
144144
if (!event) return
145145

146-
// Track click on registration button
147-
trackClick()
148-
149146
setRegistering(true)
150147
try {
151148
if (event.payment === 'Required' || event.payment === 'Paid') {

app/hackathons/[id]/page.tsx

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,7 @@ export default function HackathonDetailPage() {
122122
const { hackathon, loading: isLoading, error: fetchError } = useHackathon(slug)
123123

124124
// Track analytics
125-
const { trackClick } = useAnalyticsTracking({
125+
useAnalyticsTracking({
126126
hackathonId: slug,
127127
trackView: true,
128128
})
@@ -186,9 +186,6 @@ export default function HackathonDetailPage() {
186186
const handleRegister = async () => {
187187
if (!hackathon) return
188188

189-
// Track click on registration button
190-
trackClick()
191-
192189
setRegistering(true)
193190
try {
194191
const response = await fetch(`/api/hackathons/${slug}/register`, {

components/dashboard/AnalyticsCharts.tsx

Lines changed: 13 additions & 89 deletions
Large diffs are not rendered by default.

components/dashboard/CompanyDashboard.tsx

Lines changed: 2 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -30,17 +30,14 @@ interface CompanyDashboardStats {
3030
totalHackathons: number
3131
totalRegistrations: number
3232
totalViews: number
33-
totalClicks: number
3433
pendingApprovals: number
3534
eventMetrics: {
3635
views: number
3736
registrations: number
38-
clicks: number
3937
}
4038
hackathonMetrics: {
4139
views: number
4240
registrations: number
43-
clicks: number
4441
}
4542
recentChange?: {
4643
events: number
@@ -189,49 +186,35 @@ export function CompanyDashboard({ company }: CompanyDashboardProps) {
189186
const totalRegistrations = eventRegistrations + hackathonRegistrations
190187
/* eslint-enable @typescript-eslint/no-explicit-any */
191188

192-
// Calculate actual views and clicks from events and hackathons
189+
// Calculate actual views from events and hackathons
193190
/* eslint-disable @typescript-eslint/no-explicit-any */
194191
const eventViews = eventsData.events?.reduce(
195192
(sum: number, e: any) => sum + (e.views || 0),
196193
0
197194
) || 0
198195

199-
const eventClicks = eventsData.events?.reduce(
200-
(sum: number, e: any) => sum + (e.clicks || 0),
201-
0
202-
) || 0
203-
204196
const hackathonViews = hackathonsData.hackathons?.reduce(
205197
(sum: number, h: any) => sum + (h.views || 0),
206198
0
207199
) || 0
208-
209-
const hackathonClicks = hackathonsData.hackathons?.reduce(
210-
(sum: number, h: any) => sum + (h.clicks || 0),
211-
0
212-
) || 0
213200
/* eslint-enable @typescript-eslint/no-explicit-any */
214201

215-
// Use actual views and clicks from events/hackathons tables, not analytics
202+
// Use actual views from events/hackathons tables, not analytics
216203
const totalViews = eventViews + hackathonViews
217-
const totalClicks = eventClicks + hackathonClicks
218204

219205
setStats({
220206
totalEvents: approvedEvents.length,
221207
totalHackathons: approvedHackathons.length,
222208
totalRegistrations: totalRegistrations,
223209
totalViews: totalViews,
224-
totalClicks: totalClicks,
225210
pendingApprovals: pendingEvents.length,
226211
eventMetrics: {
227212
views: eventViews,
228213
registrations: eventRegistrations,
229-
clicks: eventClicks,
230214
},
231215
hackathonMetrics: {
232216
views: hackathonViews,
233217
registrations: hackathonRegistrations,
234-
clicks: hackathonClicks,
235218
},
236219
recentChange: {
237220
events: 0, // Could calculate from analytics
@@ -442,15 +425,6 @@ export function CompanyDashboard({ company }: CompanyDashboardProps) {
442425
{stats.eventMetrics.registrations.toLocaleString()}
443426
</span>
444427
</div>
445-
<div className="flex items-center justify-between">
446-
<div className="flex items-center gap-2">
447-
<Activity className="h-4 w-4 text-purple-400" />
448-
<span className="text-sm text-zinc-300">Clicks</span>
449-
</div>
450-
<span className="text-lg font-semibold text-white">
451-
{stats.eventMetrics.clicks.toLocaleString()}
452-
</span>
453-
</div>
454428
{stats.eventMetrics.views > 0 && (
455429
<div className="pt-2 border-t border-purple-700/30">
456430
<div className="flex items-center justify-between text-xs">
@@ -494,15 +468,6 @@ export function CompanyDashboard({ company }: CompanyDashboardProps) {
494468
{stats.hackathonMetrics.registrations.toLocaleString()}
495469
</span>
496470
</div>
497-
<div className="flex items-center justify-between">
498-
<div className="flex items-center gap-2">
499-
<Activity className="h-4 w-4 text-orange-400" />
500-
<span className="text-sm text-zinc-300">Clicks</span>
501-
</div>
502-
<span className="text-lg font-semibold text-white">
503-
{stats.hackathonMetrics.clicks.toLocaleString()}
504-
</span>
505-
</div>
506471
{stats.hackathonMetrics.views > 0 && (
507472
<div className="pt-2 border-t border-orange-700/30">
508473
<div className="flex items-center justify-between text-xs">

delete-test-company.js

Lines changed: 0 additions & 232 deletions
This file was deleted.

0 commit comments

Comments
 (0)