Skip to content

Commit 040567e

Browse files
committed
fix(react-utils): useStream and useStreamValue require dependency list input
This is to keep it compliant with react useMemo
1 parent fb887fa commit 040567e

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

libs/react-utils/src/lib/hooks/use-stream-value.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { useStream } from './use-stream';
55

66
export function useStreamValue<T>(
77
factory: FactoryOrValue<Falsy | Observable<T>>,
8-
deps: DependencyList | undefined,
8+
deps: DependencyList,
99
): T | undefined {
1010
const result = useStream(factory, deps);
1111
const value = result.status === 'success' ? result.value : undefined;

libs/react-utils/src/lib/hooks/use-stream.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ export type StreamResult<T> = PendingResult | SuccessResult<T> | ErrorResult;
1414

1515
export function useStream<T>(
1616
factory: FactoryOrValue<Falsy | Observable<T>>,
17-
deps: DependencyList | undefined,
17+
deps: DependencyList,
1818
): StreamResult<T> {
1919
const behaviorSubject$ = useMemo(() => {
2020
const behaviorSubject$ = new BehaviorSubject<StreamResult<T>>(PENDING_RESULT);

0 commit comments

Comments
 (0)