fix linter suggestions (mostly react hook dependencies)

kadaster-labs / solid-quest

Our journey on SOLID

https://labs.kadaster.nl/cases/Solid-Pods

MIT License

3 stars 1 forks source link

fix linter suggestions (mostly react hook dependencies) #80

Closed marcvanandel closed 1 year ago

marcvanandel commented 1 year ago

yarn build includes linting ... and printed some suggestions. This PR fixes those:

missing dependencies
useMemo() to reuse state between rendering

marcvanandel commented 1 year ago

😆 en het werkt ook niet ... zonder een extra 'commitje' 🙈

marcvanandel commented 1 year ago

🙈 Sterker nog, 't werkt nu helemaal niet meer goed. Ik ga deze helemaal niet mergen, maar gewoon sluiten 😏 Blijkbaar begrijp ik de React state nog onvoldoende 😁

kad-busses commented 1 year ago

Aj, dan had ik ook onvoldoende getest. Het idee was dat in het koek overzicht het lijstje niet tussendoor verdwijnt als die geupdate wordt, right? Weet eerlijk gezegd ook niet hoe dat zou kunnen, daarvoor is mijn React kennis ook onvoldoende 😢