Skip to content

Don't assume preview overflow is of length 1#291

Merged
schloerke merged 3 commits intomainfrom
safer-callbacks-invoke
Jan 10, 2023
Merged

Don't assume preview overflow is of length 1#291
schloerke merged 3 commits intomainfrom
safer-callbacks-invoke

Conversation

@cpsievert
Copy link
Copy Markdown
Contributor

Closes #290

@schloerke
Copy link
Copy Markdown
Collaborator

Failures unrelated to PR. Merging

@schloerke schloerke merged commit c96ebf5 into main Jan 10, 2023
@schloerke schloerke deleted the safer-callbacks-invoke branch January 10, 2023 17:14
schloerke pushed a commit that referenced this pull request Feb 19, 2023
* Close #290: don't assume preview overflow is of length 1

* `devtools::document()` (GitHub Actions)

* Update NEWS.md

Co-authored-by: cpsievert <cpsievert@users.noreply.github.com>
schloerke added a commit that referenced this pull request Feb 22, 2023
* main:
  Update cran-comments.md
  Update cran-comments.md
  Run revdep
  Move crashpad file to setup test file
  Update cran-comments.md
  Remove detritus Crashpad file/directory on CRAN
  Don't use github `{lintr}` or `{decor}` package
  Run revdep on 12 pkgs; ✅
  Remove C++11 from DESCRIPTION (#326)
  News entries for #284 and #288
  Load global vars when saving app objects (#307)
  Better error message when both app.R and server.R are found (#284)
  Don't assume preview overflow is of length 1 (#291)
  Force JS code to be a character for `{glue}` support (#288)
  fix typo: expect_unqiue_names -> expect_unique... (#270)
  Fix typo: use_shinytest -> use_shinytest2 (#269)
  `compare_screenshot_threshold()` should not error from `screenshot_max_difference()` (#276)
  Increment version number to 0.2.1
  Change default screenshot selector to `"scrollable_area"` (from bounding box of `"html"` DOM) (#325)
  actions/test-app; Add multi app `app-dir` support; Default upload-snapshot to `true`  (#322)
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.

Argument is of length zero error occurs inside callbacks$invoke()

2 participants