Add OAuth vignette & cache path customisation #286

hadley · 2023-08-26T20:01:08Z

Fixes #234

mgirlich · 2023-08-29T05:05:47Z

R/oauth.R

+#'
+#' @export
+#' @keywords internal
+cache_path <- function() {


I think cache_path() is too generic and other packages or custom script already use that name. So maybe rather httr2_cache_path()?

I agree that cache_path() seems a bit too short, but no other function has an httr2_ prefix. Maybe oauth_cache_path() is a better name.

And actually use it :|

#Conflicts: # NEWS.md

hadley added 4 commits August 26, 2023 14:37

Create specific OAuth vignette

a639e64

Rename cache path helper for req_cache()

f3e0e3c

Introduce new cache_path() function + env var

37dd23e

Link to vignette from docs

b8805bc

mgirlich reviewed Aug 29, 2023

View reviewed changes

hadley added 2 commits August 31, 2023 15:44

Rename to oauth_cache_path()

cae3d4e

And actually use it :|

Merge commit 'bdecf710f8369318a5f8d230b8806d22881d6d8b'

132025c

#Conflicts: # NEWS.md

hadley merged commit 5247512 into main Aug 31, 2023
12 checks passed

hadley deleted the oauth-vignette branch August 31, 2023 21:05

Provide feedback