Pawel Rucki

[Bug]: Duplicated library calls in SRC - devel deploy

1

### What happened? Some of the packages are displayed twice in the SRC. Tested devel deploy of exploratory app - Table Choices module - https://genentech.shinyapps.io/nest_exploratory_dev/ ### sessionInfo() ```R R version...

Review exported functionalities

2

Please review exported functionalities and decide which one to un-export. Please edit this post to list those object and update codebase / docs accordingly. Please watch out for docs references...

Simplify `module()` defaults

1

This is what I'm seeing in manual: ``` module( label = "module", server = function(id, ...) { moduleServer(id, function(input, output, session) { }) }, ui = function(id, ...) { tags$p(paste0("This...

### What is your question? https://github.com/insightsengineering/teal/blob/0249281cf8b2de27af235915387420c89089994c/R/modules.R#L23-L34 If so, please document. Plus it's behaviour with regard to `ui_args` and `srv_args` - in particular: a literal match of list names vs formals...

[idea] Rename `module` into `teal_module` (and also `modules` into `teal_modules`)

12

A proposal: rename `module` into `teal_module` (and `modules` into `teal_modules` accordingly). Reasons: - in line with `data` -> `teal_data` - avoid confusion internally between `modules` argument value and `modules()` function...

hidden dependencies on private functions

11

Please remove / rethink all the hidden dependencies on private functions defined elsewhere. This kind of work-around of dependencies specifications are hard to identify and maintain. https://github.com/insightsengineering/teal/blob/8ab74927a9a603ea1fe5da6398e5d0a228ccfb46/R/zzz.R#L26-L34 Some of them...

Filter mapping by module ID instead of module label

8

Conceptually, I find it quite odd to map filter to module _labels_. It should be a module _identifier_ instead. A label should be a free text but we tend to...

Soft deprecate `modules`

3

It has been a while since we switched from `modules()` into a `list()` (of `module()`). I think we should aim at deprecating this function now. That also includes references to...

### Feature description Outcome of this: https://github.com/insightsengineering/teal/issues/737#issuecomment-1291776635 As an end-user of an app I don't want to have PID, Session Info and other technical metadata displayed in my GUI **on...

### Feature description As an app developer who publish my app on shinyapps.io or RSConnect I would like to know deploy metadata such as: - deploy timestamp - last edit...