-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Add a reset button #120
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
Add a reset button #120
Conversation
Thank you very much for your work on this @thetkpark - please give me some time to set aside and review this. I also want to make sure that once we land this PR, your contribution is recognized for Hacktoberfest 2021!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fantastic work on this @thetkpark! I think there's 1 very small thing that we should fix before I can merge it in.
Please let me know if anything is unclear and thank you for your patience!
Hey @thetkpark - please let me know if you need any help with this!
Oh sorry. I have been very busy for the past 2 week. I edited the code as you suggested.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thank you very much for your help on this!
close #98.