SafetyGraphics / safetyGraphics

Clinical Trial Safety Graphics with R
https://safetygraphics.github.io/safetyGraphics/
Other
93 stars 24 forks source link

basic profile integration #703

Closed jwildfire closed 1 year ago

jwildfire commented 1 year ago

Overview

This PR adds a new tab that shows the safetyProfile shiny module as a tab in the safetyGraphics app whenever the safetyProfile package is loaded. New functionality includes:

Note that this update depends on updates to the safetyCharts package that are currently in the dev branch (to be released in v0.4)

Test Code

Checkout this branch and the latest build of safety profile and run:

remotes::install_github("safetyGraphics/safetyCharts@dev")
remotes::install_github("safetyGraphics/safetyProfile@dev")
devtools::install('safetyGraphics')

library(safetyCharts)
library(safetyProfile)
library(safetyGraphics)
safetyGraphicsApp()

Test that patProfile is optional:

unloadNamespace('safetyProfile')
safetyGraphicsApp() #profile tab should be hidden
library(safetyProfile)
safetyGraphicsApp() #profile tab should be shown
jwildfire commented 1 year ago

Next steps:

jwildfire commented 1 year ago

Last step will be to link patProfile to the charts after https://github.com/SafetyGraphics/safetyCharts/issues/125 is ready.

samussiah commented 1 year ago

per @jwildfire - make patient profile an optional plugin?

jwildfire commented 1 year ago

Added some code to make make the patient profile optional. Tab is shown when safetyProfile package is loaded and hidden otherwise. See test code in PR comments.

jwildfire commented 1 year ago

@samussiah I think this one is good to merge if you want to mark as approved and merge in dev. LMK if there are any interesting conflicts that I can help with.