Closed dpolhamus closed 6 years ago
we dont want to move to session$ns()
, right?
We can certainly try it, I think it's valid maintenance. I'd be pretty surprised if that fixes this issue though.
Actually, let's not start with that. The more we change, the more we open ourselves up to a new round of validation. Let's try to see if we can get the minimal # of changes in to get it working again and then go from there, i.e., let's target the "unnamed argments" error that's coming through right now.
There are some issues.
There is an Hmisc conflict with the newer version of R, we're going to have to use a newer Hmisc. Hopefully this doesnt wreak too much havoc. Here is a pkg dir that seems to work, with updated Hmisc and dependencies.
It looks like R:::shiny broke the way we do.call tabsetPanel. Here's the error:
It also looks like there is an "id" argument to tabsetPanel now too, I imagine we need to strip the names of the list of panels and pass them in as "id".
We can start billing work for this against the maintenance budget in August.