Consolidate `get_usage_shiny()` and `get_usage_static()` into `get_views()`

rstudio / connectapi

An R package for interacting with the RStudio Connect Server API

https://pkgs.rstudio.com/connectapi/

Other

42 stars 24 forks source link

Consolidate `get_usage_shiny()` and `get_usage_static()` into `get_views()` #252

Open nealrichardson opened 2 months ago

nealrichardson commented 2 months ago

Following the model of https://github.com/posit-dev/posit-sdk-py/pull/170, conceal from the user the fact that there are different REST APIs for different content types and just present a single way to get usage stats for all types.

jonkeane commented 2 months ago

This is the same as #218 yeah?

nealrichardson commented 2 months ago

Yep looks like it

kmasiello commented 2 months ago

Consolidating some comments from a separate slack thread...

let's seek to return one usage data frame, regardless of content type, but keep a consistent and predictable column schema. If the content type does not have relevant data, keep the column in the output but return NA.

Suggesting "usage" over "views" because the latter isn't descriptive of programmatic access to APIs or pin reads.