Skip to content

Commit 53bfa67

Browse files
authored
chore: remove unused useRequestState (#4335)
* chore: remove unused useRequestState
1 parent 04021e4 commit 53bfa67

File tree

3 files changed

+1
-111
lines changed

3 files changed

+1
-111
lines changed

ui/hooks/__tests__/common.test.tsx

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

ui/hooks/common.ts

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -16,23 +16,6 @@ export type RequestState<T> = {
1616

1717
export type ReturnType<T> = [T, boolean, RequestError, (p: Promise<T>) => void];
1818

19-
export function useRequestState<T>(): ReturnType<T> {
20-
const [state, setState] = useState<RequestState<T>>({
21-
value: null,
22-
loading: false,
23-
error: null,
24-
});
25-
26-
function req(p: Promise<T>) {
27-
setState({ ...state, loading: true });
28-
return p
29-
.then((res) => setState({ value: res, loading: false, error: null }))
30-
.catch((error) => setState({ error, loading: false, value: null }));
31-
}
32-
33-
return [state.value, state.loading, state.error, req];
34-
}
35-
3619
// Copied and TS-ified from https://usehooks.com/useDebounce/
3720
export function useDebounce<T>(value: T, delay: number) {
3821
if (process.env.NODE_ENV === "test") {

ui/index.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ import {
9191
useLinkResolver,
9292
} from "./contexts/LinkResolverContext";
9393
import { useListAutomations, useSyncFluxObject } from "./hooks/automations";
94-
import { useDebounce, useRequestState } from "./hooks/common";
94+
import { useDebounce } from "./hooks/common";
9595
import { useListEvents } from "./hooks/events";
9696
import { useFeatureFlags } from "./hooks/featureflags";
9797
import {
@@ -289,7 +289,6 @@ export {
289289
useListProviders,
290290
useListSources,
291291
useNavigation,
292-
useRequestState,
293292
useSyncFluxObject,
294293
useToggleSuspend,
295294
withBasePath,

0 commit comments

Comments
 (0)