Add sleep to ChemSpider functions

ropensci / webchem

Chemical Information from the Web

https://docs.ropensci.org/webchem

Other

160 stars 40 forks source link

Add sleep to ChemSpider functions #369

Closed stitam closed 2 years ago

stitam commented 2 years ago

This PR is related to Issue #368. ChemSpider functions did not contain any sleep, so queries were probably posted too quickly which triggered a temporary IP address blocking from the service provider. This PR adds the standard webchem_sleep(type = "API") before each ChemSpider request.

PR task list:

[x] Update NEWS
[x] Add tests (if appropriate)
[x] Update documentation with devtools::document()
[x] Check package passed

Aariq commented 2 years ago

I love that we spent the time standardizing this kind of stuff. Look how easy this fix was!

stitam commented 2 years ago

Too bad some of the query responses have changed, now we have to update the tests as well, I think I'll prioritize vcr next :)