Merge pull request #230 from R-ArcGIS/no-verify

Support querying layers that do not explicitly show query capabilities

Author: JosiahParry

Diff for: NEWS.md

@@ -1,11 +1,13 @@
 # arcgislayers (development version)
 
+
 ## New features
 
 - Improve handling of `filter_geom` by `arc_select()` by warning if applying `sf::st_union()` to the filter does not generate a length 1 sfc, or if `filter_geom` is supplied when accessing a Table, or if `filter_geom` is empty (@elipousson, #166)
 
 ## Bug fixes
 
+- Adjusts `arc_select()` to not error out when `query` capability isn't explicitly listed. Instead `cli_alert_danger()` is used to communicate the issue <https://github.com/R-ArcGIS/arcgislayers/pull/230>
 - `arc_select()` includes argument name in error message when `...` contains non-string values. <https://github.com/R-ArcGIS/arcgislayers/issues/226>
 
 ## Breaking changes

Diff for: R/arc-select.R

@@ -189,9 +189,9 @@ collect_layer <- function(
     FALSE
   )
 
-  # throw error if the layer cannot query
+  # throw warning if the layer cannot query
   if (!can_query) {
-    cli::cli_abort(
+    cli::cli_alert_danger(
       "{class(x)} {.val {x[['name']]}} does not support querying",
       call = error_call
     )