Skip to content

Commit

Permalink
Merge remote-tracking branch 'origin/1303_documentation@main' into 13…
Browse files Browse the repository at this point in the history
…03_postpone_transformer@main

* origin/1303_documentation@main:
  move the note between files
  add a note to teal_data_module constructor in its vignette about how the reactivity should be handled
  • Loading branch information
averissimo committed Oct 10, 2024
2 parents c96d333 + 4a6bbb1 commit 6768e85
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions vignettes/data-transform-as-shiny-module.Rmd
Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,9 @@ if (interactive()) {
}
```

Note: It is recommended to return `reactive()` `teal_data()` in `server` of `teal_transform_module`
as this is more robust for maintaining the reactivity of Shiny. If you are planning on using `eventReactive()` make it react to `data()`. More in [this discussion](https://github.com/insightsengineering/teal/issues/1303#issuecomment-2286239832).

### Multiple Transformers

Note that we can add multiple `teal` transformers by including `teal_transform_module` in a list.
Expand Down

0 comments on commit 6768e85

Please sign in to comment.