Skip to content

Commit a6a1056

Browse files
authored
Merge pull request #8988 from daily-co/pre-930-check-meeting-state-to-be-joined-before-calling
PRE-930 Add meetingState check to useCPULoad
2 parents d4d197a + 5e9ceb0 commit a6a1056

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

src/hooks/useCPULoad.ts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import { atom, useRecoilCallback, useRecoilValue } from 'recoil';
88

99
import { useDaily } from './useDaily';
1010
import { useDailyEvent } from './useDailyEvent';
11+
import { useMeetingState } from './useMeetingState';
1112

1213
interface CPULoad {
1314
state: DailyCpuLoadStats['cpuLoadState'];
@@ -33,6 +34,7 @@ interface Props {
3334
export const useCPULoad = ({ onCPULoadChange }: Props = {}) => {
3435
const cpu = useRecoilValue(CPULoadState);
3536
const daily = useDaily();
37+
const meetingState = useMeetingState();
3638

3739
const updateCPULoadState = useRecoilCallback(
3840
({ set, snapshot }) =>
@@ -46,7 +48,8 @@ export const useCPULoad = ({ onCPULoadChange }: Props = {}) => {
4648

4749
useEffect(() => {
4850
let mounted = true;
49-
if (!daily || daily.isDestroyed()) return;
51+
if (!daily || daily.isDestroyed() || meetingState !== 'joined-meeting')
52+
return;
5053
daily.getCpuLoadStats().then((stats) => {
5154
if (!mounted) return;
5255
updateCPULoadState({
@@ -57,7 +60,7 @@ export const useCPULoad = ({ onCPULoadChange }: Props = {}) => {
5760
return () => {
5861
mounted = false;
5962
};
60-
}, [daily, updateCPULoadState]);
63+
}, [daily, meetingState, updateCPULoadState]);
6164

6265
useDailyEvent(
6366
'cpu-load-change',

0 commit comments

Comments
 (0)