gbowne1/codebooker

[BUG] Library.jsx has some useEffect issues after today's merge(s).

gbowne1 opened this issue · 2 comments

Compiled with warnings.

[eslint]
src/components/Library/Library.jsx
Line 337:8: React Hook React.useEffect has a missing dependency: 'myRows'. Either include it or remove the dependency array. You can also do a functional update 'setMyRows(m => ...)' if you only need 'myRows' in the 'setMyRows' call react-hooks/exhaustive-deps

Search for the keywords to learn more about each warning.
To ignore, add // eslint-disable-next-line to the line before.

WARNING in [eslint]
src/components/Library/Library.jsx
Line 337:8: React Hook React.useEffect has a missing dependency: 'myRows'. Either include it or remove the dependency array. You can also do a functional update 'setMyRows(m => ...)' if you only need 'myRows' in the 'setMyRows' call react-hooks/exhaustive-deps

webpack compiled with 1 warning

Hi @gbowne1, let me resolve this error. I think it came from my last PR

@gbowne1 I have made a PR that resolves this issue. Please review.