Removed this.reset() from componentWillUnmount

[smacpherson64]

In efforts to resolve the error of #19, removing the this.reset() call in componentWillUnmount.

[smacpherson64]

This is affecting Chrome, Edge, and IE11.

Firefox appears to be working as desired.

When trying this fix, Edge no longer resets the visual.

[nukisman]

Hi there!
Why this fix is not published yet?

[smacpherson64]

Hey @nukisman, This solution unfortunately caused unwanted side effects in Edge and only partially solved the issue in other browsers. When removing this.reset() Edge no longer reset's the visual.

[nukisman]

@smacpherson64
Ooo, sadly. And what is a solution for this issue?

[smacpherson64]

Hey @nukisman, That is a good question! For the project I was on, I ended up creating a component that was a bit more hard coded to the grecaptcha api. I would prefer to use this component, but I ran out of time while debugging and had to switch. The error is coming from inside of the actual recaptcha script (not this component) when the element is removed.

At first investigation, I thought this was occurring inside of this.reset() however after implementing the fix and testing further I found it was not the core reason.