rstudio / rsconnect

Publish Shiny Applications, RMarkdown Documents, Jupyter Notebooks, Plumber APIs, and more
http://rstudio.github.io/rsconnect/
133 stars 81 forks source link

appMode override #953

Closed aronatkins closed 1 year ago

aronatkins commented 1 year ago

Allow override of the inferred appMode for situations where complicated deployments can lead to incorrectly inferred content type. For example, the presence of an app.R forces the content to be deployed as a Shiny application with the appMode = "shiny" even if that file was an auxiliary resource for some other type of content that happens to have lower precedence.

The appMode argument is accepted by:

The appDependencies function supports obtaining a set of deployment files through appFileManifest.

The appMetadata function no longer calls listDeploymentFiles. It assumes that the caller has already fully enumerated the required files and provided them in appFiles.

Fixes #948

ChaitaC commented 1 year ago

Update: caused due to dogfood migration issue. Not related to rsconnect changes.

Edge case: My deployment is stuck without any error message: Steps:

  1. remotes::install_github("rstudio/rsconnect")
  2. Create a new directory
  3. Add aap.R and quarto.qmd in the folder
  4. Deploy without specifying any appmode rsconnect::deployApp(appName = "test-app-mode", server = "rsc.radixu.com", account = "ChaitaTest")
    > rsconnect::deployApp(appName = "test-app-mode", server = "rsc.radixu.com", account = "ChaitaTest")
    ── Preparing for deployment ──────────────────────────────────────────────────
    ✔ Deploying "test-app-mode" to "server: rsc.radixu.com / username: ChaitaTest"
    ℹ Creating application on server...
    ✔ Created application with id 23056
    ℹ Bundling 2 files: app.R and quarto.qmd
    ℹ Capturing R dependencies with renv
    ✔ Found 30 dependencies
    ✔ Created 16,845b bundle
    ℹ Uploading bundle...

    Incomplete app available here: https://rsc.radixu.com/connect/#/apps/b0ce2571-aef9-4181-81b1-0ebe178cf2e7/access

ChaitaC commented 1 year ago

Test ran on ubuntu20-password

ChaitaC commented 1 year ago

Verified rsconnect

RemoteUsername: rstudio
RemoteRef: HEAD
RemoteSha: 3cd2243e602547ddfbfea1a261217081e502de9a
GithubRepo: rsconnect
GithubUsername: rstudio
GithubRef: HEAD
GithubSHA1: 3cd2243e602547ddfbfea1a261217081e502de9a
NeedsCompilation: no
Packaged: 2023-08-16 12:17:22 UTC; chaitamacpro
aronatkins commented 1 year ago

IDE work to build upon this change: https://github.com/rstudio/rstudio/pull/13510

ChaitaC commented 1 year ago

Testing instructions need update: https://github.com/rstudio/rsconnect/issues/958