use-metamask
use-metamask copied to clipboard
fix for issue #18
Heya! Thank you for your contribution! May I ask you to remove package-lock.json and dist package from the PR?
Hey! Wondering what happened to this PR, as it seems the fix is ready. Seems a shame it still can't be used with NextJS without the workaround.
Codecov Report
Merging #19 (36ae6c0) into main (1ef09b4) will decrease coverage by
1.28%
. The diff coverage is50.00%
.
@@ Coverage Diff @@
## main #19 +/- ##
===========================================
- Coverage 100.00% 98.71% -1.29%
===========================================
Files 2 2
Lines 76 78 +2
Branches 21 22 +1
===========================================
+ Hits 76 77 +1
- Misses 0 1 +1
Flag | Coverage Δ | |
---|---|---|
unittests | 98.71% <50.00%> (-1.29%) |
:arrow_down: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
src/useMetamask.js | 98.46% <50.00%> (-1.54%) |
:arrow_down: |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
I was away from maintaining the package for some time, was waiting some changes from the pr owner as well, but applying it now.