Rework box-auth functions (#109)

* Rework box-auth functions (fix #96):

- deprecate box_attach_on_auth()
- deprecate write.Renv arg to box_auth()
  - write environment variables to console instead
- refactor box_auth(), box_fresh_auth():
  - import glue and fs (both low depencency)

Also, fix typo in box_read() documentation

* Add deprecation to documentation, add is_void() to internal functions

DESCRIPTION

@@ -35,17 +35,19 @@ Imports:
     bit64,
     rio (>= 0.5.18),
     mime,
+    glue,
+    fs,
     utils,
     stats
 Suggests:
+    clipr (>= 0.3.0),
     testthat,
     knitr,
     rmarkdown,
     purrr,
     here,
-    fs,
-    glue,
-    conflicted
+    conflicted,
+    usethis
 Remotes:
     leeper/rio
 VignetteBuilder: knitr

NEWS.md

@@ -2,6 +2,13 @@
 
 ## Improvements
 
+* modifies `box_auth()` (#96):
+  - deprecates `write.Renv` arigument
+  - copies text to the clipboard, rather than overwrite the `.Renviron` file
+  - returns `invisible(NULL)` upon success 
+
+* deprecates `box_auth_on_attach()` (#96)
+
 * adds a logo (#92, @nathancday)
 
 * converts pagination method from offset to marker-based paging (#79, @awong234)

R/boxr__internal_misc.R

@@ -28,6 +28,11 @@ box_id <- function(x) {
     return(as.character(bit64::as.integer64(x)))
 }
 
+# helper to identify void values
+is_void <- function(x) {
+  is.null(x) || identical(x, "") || identical(nchar(x), 0L)  
+}
+
 
 # Function to present different package startup messages, based on whether or
 # not it looks like the user has used boxr before