maybe organize related fxns in R6 classes

ropensci / elastic

R client for the Elasticsearch HTTP API

https://docs.ropensci.org/elastic

Other

245 stars 58 forks source link

maybe organize related fxns in R6 classes #244

Closed sckott closed 3 years ago

sckott commented 5 years ago

related to #87

sckott commented 5 years ago

on classes-reorg branch (branched from r6-conn) there's one so far Cat - e.g.,

(x <- connect())
x$ping()
z <- Cat$new(x)
z
z$conn
z$help() # equiv. standalone fxn is cat_() - help is what python client uses for `cat/` route
z$aliases()
z$allocation()
z$count()
z$segments()

sckott commented 5 years ago

i think better to put this off to a later milestone and focus v1 on reorganizing around the new connection class and the big change of passing in the connection object

sckott commented 3 years ago

not going to get around to this