Skip to content

Conversation

gadenbuie
Copy link
Collaborator

Rather than use the template's directory name, we should be using the template.id for the default directory in shiny create.

Before

❯ shiny create
? Which template would you like to use? Basic app
… Creating Basic app Shiny app...
? Enter destination directory: ./01-basic-app

After

❯ shiny create
? Which template would you like to use? Basic app
… Creating Basic app Shiny app...
? Enter destination directory: ./basic-app

@gadenbuie gadenbuie requested a review from cpsievert September 4, 2024 20:11
@gadenbuie gadenbuie merged commit 1266de9 into main Sep 5, 2024
48 checks passed
@gadenbuie gadenbuie deleted the fix/shiny-create-default-dir branch September 5, 2024 15:05
schloerke added a commit that referenced this pull request Sep 5, 2024
* main:
  fix(shiny create): Use `template.id` for default directory (#1666)
schloerke added a commit to machow/py-shiny that referenced this pull request Sep 5, 2024
* main:
  chore: Fix white space formatting in `reactive.poll` Returns (posit-dev#1667)
  fix(shiny create): Use `template.id` for default directory (posit-dev#1666)
  fix(tests): Add missing env vars for shinyapps.io tests (posit-dev#1663)
  v1.1.0 release candidate
  Put streaming visual cue at the end of the content (posit-dev#1659)
  CI(debug): Print full stdout/stderr when command fails (posit-dev#1660)
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