fix: add missing live positions

Author: caieu

app/components/UI/Predict/components/PredictPicks/PredictPicksForCard.tsx

@@ -32,8 +32,8 @@ const PredictPicksForCard: React.FC<PredictPicksForCardProps> = ({
 }) => {
   const { data: fetchedPositions = [] } = usePredictPositions({
     marketId,
-    refetchInterval: positionsProp ? undefined : 10000,
     enabled: !positionsProp,
+    livePriceUpdates: !positionsProp,
   });
 
   const basePositions = positionsProp ?? fetchedPositions;

app/components/UI/Predict/components/PredictPositionsHeader/PredictPositionsHeader.tsx

@@ -94,7 +94,10 @@ const PredictPositionsHeader = forwardRef<
   const evmAccount = getEvmAccountFromSelectedAccountGroup();
   const selectedAddress = evmAccount?.address ?? '0x0';
   const { isDepositPending } = usePredictDeposit();
-  const { data: activePositions } = usePredictPositions({ claimable: false });
+  const { data: activePositions } = usePredictPositions({
+    claimable: false,
+    livePriceUpdates: true,
+  });
   const { data: claimablePositions = [] } = usePredictPositions({
     claimable: true,
   });

app/components/UI/Predict/components/PredictSportCardFooter/PredictSportCardFooter.tsx

@@ -53,7 +53,7 @@ const PredictSportCardFooter: React.FC<PredictSportCardFooterProps> = ({
   const { data: positions = [], isLoading } = usePredictPositions({
     marketId: market.id,
     claimable: false,
-    refetchInterval: 10000,
+    livePriceUpdates: true,
   });
 
   const { data: claimablePositions = [] } = usePredictPositions({

app/components/Views/Homepage/Sections/Predictions/hooks/usePredictPositionsForHomepage.test.ts

@@ -3,6 +3,7 @@ import { usePredictPositionsForHomepage } from './usePredictPositionsForHomepage
 import type { PredictPosition } from '../../../../../UI/Predict/types';
 
 const mockRefetch = jest.fn().mockResolvedValue(undefined);
+const mockUsePredictPositions = jest.fn();
 let mockUsePredictPositionsReturn: {
   data: PredictPosition[] | undefined;
   isLoading: boolean;
@@ -16,7 +17,12 @@ let mockUsePredictPositionsReturn: {
 };
 
 jest.mock('../../../../../UI/Predict/hooks/usePredictPositions', () => ({
-  usePredictPositions: () => mockUsePredictPositionsReturn,
+  usePredictPositions: (
+    ...args: Parameters<typeof mockUsePredictPositions>
+  ) => {
+    mockUsePredictPositions(...args);
+    return mockUsePredictPositionsReturn;
+  },
 }));
 
 const createMockPosition = (id: string, currentValue = 12): PredictPosition =>
@@ -36,6 +42,7 @@ const createMockPosition = (id: string, currentValue = 12): PredictPosition =>
 describe('usePredictPositionsForHomepage', () => {
   beforeEach(() => {
     jest.clearAllMocks();
+    mockUsePredictPositions.mockClear();
     mockUsePredictPositionsReturn = {
       data: [
         createMockPosition('1'),
@@ -157,6 +164,28 @@ describe('usePredictPositionsForHomepage', () => {
       expect(result.current.totalClaimableValue).toBe(0);
     });
 
+    it('enables live updates for active positions', () => {
+      renderHook(() => usePredictPositionsForHomepage({ claimable: false }));
+
+      expect(mockUsePredictPositions).toHaveBeenCalledWith(
+        expect.objectContaining({
+          claimable: false,
+          livePriceUpdates: true,
+        }),
+      );
+    });
+
+    it('disables live updates for claimable positions', () => {
+      renderHook(() => usePredictPositionsForHomepage({ claimable: true }));
+
+      expect(mockUsePredictPositions).toHaveBeenCalledWith(
+        expect.objectContaining({
+          claimable: true,
+          livePriceUpdates: false,
+        }),
+      );
+    });
+
     it('treats undefined currentValue as 0 in totalClaimableValue sum', () => {
       mockUsePredictPositionsReturn.data = [
         {

app/components/Views/Homepage/Sections/Predictions/hooks/usePredictPositionsForHomepage.ts

@@ -32,7 +32,7 @@ export const usePredictPositionsForHomepage = (
   const { data, isLoading, error, refetch } = usePredictPositions({
     claimable,
     enabled,
-    livePriceUpdates: true,
+    livePriceUpdates: !claimable,
   });
 
   const allPositions = useMemo(() => data ?? [], [data]);