-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
This is good stuff, just needs a small fix #4
Comments
Hi @fermmm! Thank you for openings this issue. Please go ahead and open a pull request addressing this. Happy to review, accept, and merge it for you. |
Thanks for the fix! |
any explanation why those lines fix the issue? |
This is a very simple package and the code quality is good, also works great after applying a fix.
Problems:
These are small problems and easy to fix.
Solution:
Change line 19:
const [lastValue, setLastValue] = useState<boolean>(false);
for this:
const [lastValue, setLastValue] = useState<boolean>(null);
Remove line 29:
setLastValue(false);
The package is only 1 file so if the maintainer doesn't fix it anyone can just copy the code into a file with no problems.
The text was updated successfully, but these errors were encountered: