|
1 | | -import { useEffect } from 'react' |
2 | 1 | import { CheckCircle, XCircle } from 'lucide-react' |
3 | 2 | import type { SuccessScreenProps } from '../../types/sender' |
4 | 3 | import { trackTransferComplete } from '../../lib/analytics' |
@@ -50,11 +49,13 @@ export function TransferSuccessScreen({ metadata, onDone }: SuccessScreenProps) |
50 | 49 | const wasStopped = metadata.wasStopped || false |
51 | 50 | const isReceiver = !!metadata.downloadPath |
52 | 51 |
|
53 | | - useEffect(() => { |
54 | | - // Track analytics for all transfers, regardless of completion status or file size |
55 | | - // This provides better insights into user behavior and transfer patterns |
56 | | - trackTransferComplete(metadata.fileSize, isReceiver ? 'receiver' : 'sender', wasStopped) |
57 | | - }, [isReceiver, metadata.fileSize, wasStopped]) |
| 52 | + const handleDone = () => { |
| 53 | + // Only track analytics if transfer completed successfully (not stopped) |
| 54 | + if (!wasStopped) { |
| 55 | + trackTransferComplete(metadata.fileSize, isReceiver ? 'receiver' : 'sender') |
| 56 | + } |
| 57 | + onDone() |
| 58 | + } |
58 | 59 |
|
59 | 60 | return ( |
60 | 61 | <div className="flex flex-col items-center justify-center space-y-6 "> |
@@ -179,7 +180,7 @@ export function TransferSuccessScreen({ metadata, onDone }: SuccessScreenProps) |
179 | 180 |
|
180 | 181 | {/* Done Button */} |
181 | 182 | <button |
182 | | - onClick={onDone} |
| 183 | + onClick={handleDone} |
183 | 184 | className="w-full max-w-sm py-3 px-6 rounded-md font-medium transition-colors focus:outline-none focus:ring-2 focus:ring-offset-2" |
184 | 185 | style={{ |
185 | 186 | backgroundColor: 'var(--app-primary)', |
|
0 commit comments