avishai84 / numbers

0 stars 0 forks source link

useEffect has no meaning #5

Closed eranmore closed 4 years ago

eranmore commented 4 years ago

https://github.com/avishai84/numbers/blob/39ce4d0c20109a2428dc40f1f9256a492e0caa14/src/components/Audio.jsx#L39

why you don't set to true as initial value and only in useEffect ? also, the dependencies of useEffect are not relevant

eranmore commented 4 years ago

opened with incorrect user