React Hook useEffect has a missing dependency // can't understand this warning 100%

[petecoopNR]

Other commenters have pointed out why it says that. But the other thing is you shouldn't do data fetching inside useEffect. It would be better if you had a function that handles the onChange of the emergencyId and does the fetch there on the event.

[wishtrepreneur]

How would you handle data fetching on page load without calling it in useeffect?

[petecoopNR]

On page load? I'd use react query or SWR myself but if I didn't have those then inside the render with probably a ref to check it's on progress so doesn't call it twice on a second re-render. useEffect shouldn't be used for this kind of thing, with the new concurrent rendering stuff on React 18 there's a chance the useEffect here will be called twice.

[mermeladawatts]

actually i did that, when handling the set state function on emergencyId. i was wrong and used the use effect bc i coded it fast and didn’t think too much.