This PR moved all functions that earlier were put in ./inst/app and ./inst/app/R into ./R. This way, all internal functions should be available within the apps ui() and server() functions. To align with best practices for R packages and remove notes and warnings from R CMD check, several changes has been made:
The global.R file has been removed and it's functions has been added elsewhere in the code, including in the server() and run_pioneer() functions
All code from conditionalUI.R has been added to server(). The code for the server() function can be split into Shiny modules in a later PR
All calls to require() has been replaced with @import or @importFrom statements or the use of ::. The package now includes far more packages in the Imports section to make sure that all packages needed to run the app are available
This PR does not include new tests. New test should be written for most functions. However, the purpose of this PR is to move all functions out of ./inst/app. We should have a review of how functions can be optimised now that they are all internal functions. This should be a separate PR where we also introduce new tests.
This PR moved all functions that earlier were put in
./inst/appand./inst/app/Rinto./R. This way, all internal functions should be available within the appsui()andserver()functions. To align with best practices for R packages and remove notes and warnings from R CMD check, several changes has been made:global.Rfile has been removed and it's functions has been added elsewhere in the code, including in theserver()andrun_pioneer()functionsconditionalUI.Rhas been added toserver(). The code for theserver()function can be split into Shiny modules in a later PRrequire()has been replaced with@importor@importFromstatements or the use of::. The package now includes far more packages in the Imports section to make sure that all packages needed to run the app are availableThis PR does not include new tests. New test should be written for most functions. However, the purpose of this PR is to move all functions out of
./inst/app. We should have a review of how functions can be optimised now that they are all internal functions. This should be a separate PR where we also introduce new tests.