Correctly return TRUE is user trust extension author

fixes #212
quarto-dev · Sep 16, 2024 · 0fcb998 · 0fcb998
1 parent 3c46699
commit 0fcb998
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 1 deletion.
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -1,6 +1,6 @@
 Package: quarto
 Title: R Interface to 'Quarto' Markdown Publishing System
-Version: 1.4.4.9002
+Version: 1.4.4.9003
 Authors@R: c(
     person("JJ", "Allaire", , "jj@posit.co", role = "aut",
            comment = c(ORCID = "0000-0003-0174-9868")),

diff --git a/NEWS.md b/NEWS.md
@@ -6,6 +6,8 @@
 
 - `quarto_path()` now returns a normalized path with potential symlink resolved, for less confusion with `quarto_binary_sitrep()` (thanks, @jennybc).
 
+- Fix an error with interactive prompt for extension approval (thanks, @wjschne, #212).
+
 # quarto 1.4.4
 
 - `quarto_preview()` now looks at `quarto preview` log to browse to the correct url when inside RStudio viewer (thanks, @aronatkins, #167).

diff --git a/R/utils-prompt.R b/R/utils-prompt.R
@@ -18,6 +18,8 @@ check_extension_approval <- function(no_prompt = FALSE, what = "Something", see_
     if (!prompt_value %in% "y") {
       cli::cli_inform("{what} not installed.")
       return(invisible(FALSE))
+    } else {
+      return(invisible(TRUE))
     }
   }
 }