trigger an error if the api script is not loaded

[andrepolischuk]

also I rewrited mounting the api script with global Promise to fix case with multiple captchas. if multiple captchas are using, handleOnLoad callback is pushed to the onLoadListeners only for first component. next captchas in componentDidMount have apiScriptRequested true, so handleOnLoad is not adding to callback's array

[andrepolischuk]

@zoryana94 @DSergiu hi, could you take a look at this pr?

[andrepolischuk]

@zoryana94 @DSergiu ping

[brdlyptrs]

Hi @andrepolischuk thanks for the PR. Looks pretty good overall, I've added a few comments above.

[brdlyptrs]

One last note, looks good!

Just to confirm, as it looks like it, this update will handle multiple hCaptcha components loading at the same time waiting for the script tag to load correct?

[andrepolischuk]

Just to confirm, as it looks like it, this update will handle multiple hCaptcha components loading at the same time waiting for the script tag to load correct?

yes. first instance of the component mounts the script and waits promise (in module root) to handle onload. the rest of instances check mounted script in dom, and wait the same promise to handle onload.

[brdlyptrs]

I think we can bump this version to 1.3.1 and then merge.

[andrepolischuk]

@brdlyptrs I pulled latest master into this branch

[andrepolischuk]

@brdlyptrs are you planning to publish this soon?

[zoryana94]

@brdlyptrs are you planning to publish this soon?

Hi @andrepolischuk,
I'm going to merge it today. We'll probably need your assistance in testing.
Thank you a lot for your work! And sorry that it took a bit longer than expected.

[brdlyptrs]

We can version bump separately.