Skip to content

Enable error console when running locally. #1060

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 5 commits into from
Mar 26, 2024
Merged

Enable error console when running locally. #1060

merged 5 commits into from
Mar 26, 2024

Conversation

wch
Copy link
Collaborator

@wch wch commented Jan 24, 2024

This PR enables the error console when running shiny apps locally.

When apps are run locally, they are launched with shiny run; when they are run on a server (like Connect) they are launched with uvicorn, and in that case the error console will not be enabled.

@wch wch added this to the v0.8.0 milestone Jan 24, 2024
@wch wch force-pushed the dev-mode-error-console branch from b93a1ff to a65ae2c Compare January 24, 2024 20:47
@wch wch force-pushed the dev-mode-error-console branch from a65ae2c to cf26fa3 Compare January 24, 2024 21:39
@jcheng5
Copy link
Collaborator

jcheng5 commented Mar 1, 2024

I wonder if we should make it based on --reload instead? I was thinking about suggesting that we change the way Connect and SSOS load Shiny apps, I think there are plenty of other ways we could have them inject their middleware than to load the Shiny app object and wrap it.

@wch wch modified the milestones: v0.8.0, v0.9.0 Mar 1, 2024
@jcheng5
Copy link
Collaborator

jcheng5 commented Mar 26, 2024

After discussion with @wch in person, the important thing we want to ensure is that Connect never shows the error console by default, so we should write a automated test that verifies that and this PR can be merged as-is.

@wch wch merged commit 3188b86 into main Mar 26, 2024
@wch wch deleted the dev-mode-error-console branch March 26, 2024 21:48
schloerke added a commit that referenced this pull request Mar 27, 2024
* main:
  feat: Create `Session.set_message_handler(name, handler)` (#1253)
  Fix input name
  Enable error console when running locally. (#1060)
  test: Update test to use variable, not capture stdout (#1236)
schloerke added a commit that referenced this pull request Mar 28, 2024
* main:
  test(CI): Add merge queue support for faster PR testing (#1265)
  fix(panel_conditional): Include `.shiny-panel-conditional` class (#1257)
  feat: Create `Session.set_message_handler(name, handler)` (#1253)
  Fix input name
  Enable error console when running locally. (#1060)
  test: Update test to use variable, not capture stdout (#1236)
  test: Fix deploys for shinyapps.io and connect (#1224)
  chore(test): Playwright test refactors and clean up (#1204)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants