Fix for when localStorage is available but access is restricted

[chaitanyapotti]

No description provided.

[coveralls]

Coverage decreased (-0.1%) to 83.02% when pulling 626bcdc on chaitanyapotti:1.x into ce27738 on ethereum:1.x.

[miguelmota]

Great fix @chaitanyapotti! Hope this gets merged

[alcuadrado]

Hey @chaitanyapotti! Thanks for this PR.

Can you explain a little what it does, and especially, how (the error's checks)? It'd be great to have that info in the code.

[chaitanyapotti]

@alcuadrado Here is the relevant documentation

[alcuadrado]

Thanks for that link @chaitanyapotti. Do you mind adding it as a comment in the function?

[chaitanyapotti]

@nivida Hey, any timeline on when this will be released?

[miguelmota]

Is this feature detection available in the latest npm package release yet?

[smithki]

I'm wondering the same. Is there a timeline on this release? (or is it already available?)

EDIT: I confirmed for myself that the code is not yet released. This is blocking me on Brave support right now 😢

[miguelmota]

I ended up using ethers.js which is much more lightweight and doesn't have issues like this