search

ctsit / redcapcustodian

Simplified, automated data management on REDCap systems
Other
12 stars 6 forks source link

Pass DBI connection object to `write_summary_metrics` #120

Closed pbchase closed 12 months ago

pbchase commented 1 year ago

I feel like we should have passed the DBI connection object to write_summary_metrics instead of doing this:

https://github.com/ctsit/redcapcustodian/blob/43180b78d3c9eef85d947c55a9483a422b5aa2df/R/summary_metrics.R#L48

As I go to use write_summary_metrics in novel ways, testing is tricker because its harder to control the connection object it writes to.

@ChemiKyle, what do you think?

ChemiKyle commented 1 year ago

121 details the approach we discussed today

ChemiKyle commented 12 months ago

Closed by #122