pharmaR / riskassessment

Risk Assessment Demo App: https://rinpharma.shinyapps.io/riskassessment
https://pharmar.github.io/riskassessment/
Other
98 stars 26 forks source link

Switch Report Configs storage to DB #661

Closed AARON-CLARK closed 2 weeks ago

AARON-CLARK commented 9 months ago

Why use txt files?

I don't think the app deployer should have to learn about & specify yet another folder/file system in inst/db_config.yml for storing these preferences. It would seem prudent to leverage our user-based credentials db to store this information, no? There could be a table to store org-level and a table for user-level preferences. Thoughts?

Originally posted by @AARON-CLARK in https://github.com/pharmaR/riskassessment/pull/645#pullrequestreview-1680112464

aclark02-arcus commented 2 weeks ago

this issue is stale and this feature never really hit master, at least from the perspective of using txt files to execute the proposed functionality in #645.