Closed jwildfire closed 1 year ago
cc: @samussiah @bzkrouse
@samussiah Could you do an initial review this PR sometime this week and let me know what you think? Would like to get back to this and make the UI updates next week.
Going to try to avoid using shinyjs in tab titles via this approach
@bzkrouse @samussiah Ended up doing a fairly significant refactor of the app to simplify the status workflow. Still planning to add a bit more detail and clean up checks etc.
Think I'm pretty much done with this. Kept the UI simple for now (added a status icon with some details on mouseover) - can consider making it more complex in a future release (Modal? Separate collapsable Shiny Module?).
@samussiah Just bumping this to put it back on your radar.
Overview
checkChartStatus()
is in good shape and has some simple unit tests.Test Code
See
tests/checkChartStatus.R
for functional testing. See code below for updates in UI.