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

Republishing to RPubs is broken from RStudio #1016

Merged
merged 4 commits into from
Oct 12, 2023
Merged

Conversation

jcheng5
Copy link
Member

@jcheng5 jcheng5 commented Oct 12, 2023

Fixes #976. See #976 (comment) for explanation.

@aronatkins
Copy link
Contributor

Thanks, @jcheng5! I'll take it from here, fix the lint complaints and add NEWS.

@aronatkins
Copy link
Contributor

I've confirmed that rpubs republishing is fixed with this change.

Unfortunately, the off-by-one deployment record means that existing documents are "stuck".

Folks can either use rsconnect::forgetDeployment() or manually adjust the rsconnect deployment record.

Assuming I'm publishing from an index.Rmd file, the deployment record lives at:
rsconnect/documents/index.Rmd/rpubs.com/rpubs/Document.dcf

An example "bad" record:

name: Document
title:
username:
account: rpubs
server: rpubs.com
envVars: rpubs.com
hostUrl: https://api.rpubs.com/api/v1/document/1097969/984f9439141d43f4a35ae663a972522e
appId: https://api.rpubs.com/api/v1/document/1097969/984f9439141d43f4a35ae663a972522e
bundleId: http://rpubs.com/publish/claim/1097969/4ef82fe6cb3f41f39e42de23caf3e806
url: 1697138036.07503
version: 1

An example "good" record:

name: Document
title:
username:
account: rpubs
server: rpubs.com
hostUrl: rpubs.com
appId: https://api.rpubs.com/api/v1/document/1097975/bf849d4b51884d1a9716e7028e5b33a6
bundleId: https://api.rpubs.com/api/v1/document/1097975/bf849d4b51884d1a9716e7028e5b33a6
url: http://rpubs.com/publish/claim/1097975/e57795f1581b4f30a16c353b4ae7c911
version: 1

There were two mistakes:

  1. an envVars argument was not added
  2. the when and lastSyncTime arguments were removed.

@aronatkins aronatkins requested a review from hadley October 12, 2023 19:31
@aronatkins aronatkins merged commit 525289d into main Oct 12, 2023
@aronatkins aronatkins deleted the reset-on-rpubs-update branch October 12, 2023 20:14
@jcheng5
Copy link
Member Author

jcheng5 commented Oct 13, 2023

Thanks @aronatkins!

@JBell39
Copy link

JBell39 commented Oct 25, 2023

I don't know what I am doing wrong. Here is my updated dcf:

name: Document
title:
username:
account: rpubs
server: rpubs.com
hostUrl: rpubs.com
appId: https://api.rpubs.com/api/v1/document/1099849/f24f13c92bd4430a8e6d83d25154f1fc
bundleId: http://rpubs.com/publish/claim/1099849/37d672b76e4748539e6d5983395371e5
url:  https://rpubs.com/bellj39/1099849
version: 1

@JBell39
Copy link

JBell39 commented Oct 25, 2023

I don't know what I am doing wrong. Here is my updated dcf:

name: Document
title:
username:
account: rpubs
server: rpubs.com
hostUrl: rpubs.com
appId: https://api.rpubs.com/api/v1/document/1099849/f24f13c92bd4430a8e6d83d25154f1fc
bundleId: http://rpubs.com/publish/claim/1099849/37d672b76e4748539e6d5983395371e5
url:  https://rpubs.com/bellj39/1099849
version: 1

NM! I friggin' figured it out. lol. Thank you!

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.

Republishing to RPubs, Error: HTTP/2 stream 1 was reset
3 participants