ctsit / redcapcustodian

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

Add basic sync_table function #47

Closed ChemiKyle closed 2 years ago

ChemiKyle commented 2 years ago

Closes #24

After struggling for too long with manufacturing a SQL statement for UPDATEs (espec with needing to do something like .data${{primary_key}}), I found the dbx package.

pbchase commented 2 years ago

There are no tests. It makes me sad. :-(

pbchase commented 2 years ago

FYI, I added dbx to https://github.com/ctsit/rstudio-dockerfile to the https://github.com/ctsit/rstudio-dockerfile/pkgs/container/rstudio-ci could run the tests

pbchase commented 2 years ago

FYI, I am adding a test that uses dataset_diff_test_user_data as input.

ChemiKyle commented 2 years ago

Oh, I was going to add tests along with hacking in some logging of dbx's actions, but I'll take yours too.

pbchase commented 2 years ago

I am done adding my test. Build check passes. I feel like it's time to merge. Do you agree?