Skip to content

Commit b0a36b8

Browse files
authored
Use useSetAtom in SocketProvider for write-only atom (#1498)
## Summary - Replaced write-only `useAtom` usage with `useSetAtom` for `fileTransferProgressAtom` in `SocketProvider`, so the provider does not subscribe to atom value updates. Closes #1493. ## Test plan - [x] CI lint passes - [x] Smoke-test remote file transfer / socket progress UI if available Made with [Cursor](https://cursor.com)
2 parents 486d187 + 5666cb7 commit b0a36b8

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

src/libs/SocketProvider.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
/* eslint-disable no-console */
66
import React, { ReactNode, createContext, useEffect } from 'react';
77
import { Socket, io } from 'socket.io-client';
8-
import { useAtom } from 'jotai';
8+
import { useSetAtom } from 'jotai';
99
import { getOrCreateInstanceId } from './axiosInstance';
1010
import { fileTransferProgressAtom } from '../store/app';
1111
import { FileProgress, FileStatus } from '../model/APIData';
@@ -24,7 +24,7 @@ interface SocketProviderProps {
2424
}
2525

2626
export const SocketProvider: React.FC<SocketProviderProps> = ({ children }) => {
27-
const [_, setFileTransferProgress] = useAtom(fileTransferProgressAtom);
27+
const setFileTransferProgress = useSetAtom(fileTransferProgressAtom);
2828
const instanceId = getOrCreateInstanceId();
2929

3030
useEffect(() => {

0 commit comments

Comments
 (0)