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

poll_task is trowing a 404 error because of changes in api reference #338

Closed
francoisluc opened this issue Nov 25, 2024 · 3 comments · Fixed by #339
Closed

poll_task is trowing a 404 error because of changes in api reference #338

francoisluc opened this issue Nov 25, 2024 · 3 comments · Fixed by #339
Assignees

Comments

@francoisluc
Copy link

I used the documented example in posit documentation and got a 404 error.
Image

library(connectapi)

CONTENT_GUID <- "34e425fb-179c-48e8-81bd-b334eb449f55"

client <- connect()
content <- content_item(client, CONTENT_GUID)
render_task <- content_render(content)

poll_task(render_task)

If you change this line
https://github.com/rstudio/connectapi/blame/f6093432582e1bed9ebcadbf613944cd691e5ead/R/deploy.R#L583

to

  task_data <- con$task(task$task$id, wait = wait, first = first)

It solves the problem. I saw some changes to the tasks endpoint in august and I think this code was not updated as id parameter is now mandatory.

Thanks in advance for fixing this small bug !

@toph-allen toph-allen self-assigned this Nov 27, 2024
@toph-allen
Copy link
Collaborator

@francoisluc Thanks for bringing this to our attention!

@toph-allen
Copy link
Collaborator

@francoisluc This is fixed in the dev version and will be in the next release.

@francoisluc
Copy link
Author

@toph-allen Thanks for the quick fix, this is awesome.

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 a pull request may close this issue.

2 participants