Skip to content

Commit fc6b3f7

Browse files
committed
better error messages when api key is missing, asking users to use the new set_api_key function instead of doing it manually.
1 parent ed3dc3a commit fc6b3f7

4 files changed

Lines changed: 9 additions & 12 deletions

File tree

R/cancensus.R

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -95,8 +95,7 @@ get_census <- function (dataset, regions, level=NA, vectors=c(), geo_format = NA
9595
digest::digest(param_string, algo = "md5"), ".rda")
9696
if (!use_cache || !file.exists(data_file)) {
9797
if (!have_api_key) {
98-
stop(paste("No API key set. Use Sys.setenv(CM_API_KEY = '<your API key>') or",
99-
"options(cancensus.api_key = '<your API key>') to set one."))
98+
stop(paste("No API key set. Use set_api_key('<your API ket>`) to set one, or set_api_key('<your API ket>`, install = TRUE) to save is permanently in our .Renviron."))
10099
}
101100
url <- paste0(base_url, "data.csv")
102101
response <- if (!quiet) {
@@ -147,8 +146,7 @@ get_census <- function (dataset, regions, level=NA, vectors=c(), geo_format = NA
147146
geo_file <- cache_path(geo_base_name, ".geojson")
148147
if (!use_cache || !file.exists(geo_file)) {
149148
if (!have_api_key) {
150-
stop(paste("No API key set. Use Sys.setenv(CM_API_KEY = '<your API key>') or",
151-
"options(cancensus.api_key = '<your API key>') to set one."))
149+
stop(paste("No API key set. Use set_api_key('<your API ket>`) to set one, or set_api_key('<your API ket>`, install = TRUE) to save is permanently in our .Renviron."))
152150
}
153151
url <- paste0(base_url, "geo.geojson")
154152
response <- if (!quiet) {

R/intersect_geometry.R

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ get_intersecting_geometries <- function(dataset, level, geometry, simplified = F
4545
use_cache = TRUE, quiet = FALSE,
4646
api_key=Sys.getenv("CM_API_KEY")) {
4747
api_key <- robust_api_key(api_key)
48-
have_api_key <- !is.null(api_key)
48+
have_api_key <- valid_api_key(api_key)
4949
result <- NULL
5050

5151
if ("sf" %in% class(geometry)) {
@@ -74,8 +74,7 @@ get_intersecting_geometries <- function(dataset, level, geometry, simplified = F
7474

7575
if (!use_cache || !file.exists(data_file)) {
7676
if (!have_api_key) {
77-
stop(paste("No API key set. Use options(cancensus.api_key = 'XXX') or",
78-
"Sys.setenv(CM_API_KEY = 'XXX') to set one."))
77+
stop(paste("No API key set. Use set_api_key('<your API ket>`) to set one, or set_api_key('<your API ket>`, install = TRUE) to save is permanently in our .Renviron."))
7978
}
8079
url <- paste0(cancensus_base_url(),"/api/v1/intersecting_geographies")
8180
body <- list(dataset=dataset,

docs/articles/Making_maps_with_cancensus.html

Lines changed: 4 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

docs/pkgdown.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ articles:
88
cancensus: cancensus.html
99
data_discovery: data_discovery.html
1010
intersecting_geometries: intersecting_geometries.html
11-
last_built: 2021-01-24T07:54Z
11+
last_built: 2021-01-25T04:15Z
1212
urls:
1313
reference: https://mountainmath.github.io/cancensus/index.html/reference
1414
article: https://mountainmath.github.io/cancensus/index.html/articles

0 commit comments

Comments
 (0)