Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dev webtest comptox #273

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Conversation

andschar
Copy link
Contributor

Pull Request

PR to add the webtest_query() function to webchem. This PR replaces the erroneous and now closed PR #251. Please also mind the discussion we already had here.


PR task list:

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

@andschar
Copy link
Contributor Author

the checks currently fail, because the web-service seems to behave differently compared to some weeks ago. I wrote them a mail and will implement changes according to their response.

@andschar andschar marked this pull request as draft July 15, 2020 10:42
@stitam stitam added this to the RC2019F milestone Sep 5, 2020
@stitam stitam removed this from the RC2019F milestone Sep 25, 2020
@stitam stitam self-assigned this Mar 1, 2021
@stitam stitam force-pushed the dev_webtest_comptox branch from 265deed to 437f150 Compare March 1, 2021 21:51
@stitam stitam force-pushed the dev_webtest_comptox branch from 80075a6 to 0bb8a1d Compare March 2, 2021 06:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants