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

insertUI() now supports execution of <script> #3630

Merged
merged 4 commits into from
May 10, 2022
Merged

insertUI() now supports execution of <script> #3630

merged 4 commits into from
May 10, 2022

Conversation

cpsievert
Copy link
Collaborator

@cpsievert cpsievert commented May 9, 2022

Closes #1545

@cpsievert cpsievert requested a review from schloerke May 9, 2022 15:48
cpsievert added a commit to rstudio/shinycoreci-apps that referenced this pull request May 9, 2022
Copy link
Collaborator

@schloerke schloerke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Up to you if you'd like to fold in the replace case to the switch

schloerke added a commit to rstudio/shinycoreci that referenced this pull request May 10, 2022
Co-Authored-By: Carson Sievert <cpsievert1@gmail.com>
@wch wch changed the title insertUI() now uses jQuery to insert HTML; and thus, now supports execution of <script> insertUI() now supports execution of <script> May 10, 2022
@wch wch merged commit 78d77ce into main May 10, 2022
@wch wch deleted the insert-ui-jquery branch May 10, 2022 16:43
schloerke added a commit to rstudio/shinycoreci that referenced this pull request May 10, 2022
Co-authored-by: Carson Sievert <cpsievert1@gmail.com>
schloerke added a commit to rstudio/shinycoreci that referenced this pull request May 10, 2022
* main:
  Add app to test rstudio/shiny#3630 (#96)
  Display logs in website; Hide logs on first pass;  Remove `view_test_diff()` and `view_test_results()` (#97)
  do not destroy non-pkgdown files
  Make sure progress bar shows up on GHA
  update readme
  ignore artifacts from other branches
  Add link to test results
  Merge branch 'main' of github.com:rstudio/shinycoreci
  Comment warning check as some warning is being cause somewhere outside of our control
  shinytest2-chromote snapshots review - 6
  shinytest2-chromote snapshots review - 5
  Change from fixed time sleep to `$wait_for_idle()` where possible (#95)
  Fix repo location and user running action
  typo
  add specific method for 'trigger_tests()'
  Accept all snaps for fix_snaps(); Fix apps heights and widths. Remove 134 snaps(#94)
  Generate apps deps (GitHub Actions)
  shinytest2-chromote snapshots review - 4
  Fix a handful of apps to create consistent screenshots (#93)
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.

When inserting javascript with insertUI, does it have to be wrapped in tags$head()? (Bug or feature?)
3 participants