update vetiver deployment

[gsingh91]

Updating vetiver deployment to Connect to use vetiver native function.

@SamEdwardes I am having issues testing the deployment from Colorado Workbench, but I was able to test it locally on open source Desktop. Can you please test it once in your workbench env and merge? Thanks!

[gsingh91]

@SamEdwardes done. Please review.

[juliasilge]

Am I understanding correctly that when you have a script running on Connect that deploys an API to the same Connect server, you need to specify a user in that script?

[SamEdwardes]

Thanks Gagan - this is on my to-do list.

---

@juliasilge

Am I understanding correctly that when you have a script running on Connect that deploys an API to the same Connect server, you need to specify a user in that script?

I am interested in your input here, but I think we do:

• Vetiver deploys the API using the rsconnect::deployApp() function.
• For the deployApp function to work it relies on two things:
  • You have already added a server with rsconnect::addServer()
  • You have already added a user with rsconnect::connectApiUser() or rsconnect::connectUser()
• The user publishing the content has no control over what is happening on the Connect server other than the code that gets executed in this script, therefore they have to add the server and user in this script.

However, I would prefer if we did not have to do this and could publish without having to run the add functions. For example:

vetiver_deploy_rsconnect(
  connect_server = "https://connect.example.com",
  connect_api_key = Sys.getenv("CONNECT_API_KEY"),
  board = board,
  name = pin_name,
  version = NULL,
  launch.browser = FALSE,
  appTitle = "Bike Predict - Model - API",
  predict_args = list(debug = FALSE),
)

[SamEdwardes]

@SamEdwardes done. Please review.

I did some testing and it looks like everything is working. I made a few small updates:

• Hardcode in the appID to the production API endpoint
• Deleted code that either commented out or no longer required based on our new deployment method

[juliasilge]

I agree that it would be better if rsconnect (the R package) knew to look up the server and user if something like deployApp() was being executed on Connect itself.