pre-release vignettes review

[kartikeyakirar]

PR to #640

[vedhav]

I was testing if all the apps fro the vignettes worked after the modules as a variable change.
All the example apps worked except for the example in the vignette using-scatterplot-matrix.Rmd
Error:

Listening on http://127.0.0.1:7784
Browsing http://127.0.0.1:7784
[INFO] 2024-02-27 20:09:27.6905 pid:97964 token:[6b67bbd1] teal Initializing reporter_previewer_module
Warning: Error in teal.transform::data_extract_ui: more than one dataset in data_extract_spec but is_single_dataset parameter is set to TRUE
  3: runApp
  2: print.shiny.appobj
  1: <Anonymous>

[kartikeyakirar]

I was testing if all the apps fro the vignettes worked after the modules as a variable change.
All the example apps worked except for the example in the vignette using-scatterplot-matrix.Rmd

@vedhav, I reviewed the tm_g_scatterplotmatrix function and noticed that it uses teal.transform::is_single_dataset on the variables parameter. In the case of the "Multiple wide datasets" module configuration, we currently utilize ADSL twice, whereas previously we used ADSL2. The teal.transform::is_single_dataset function checks the data names in the variables parameter and identifies only a single name, "ADSL," which leads to this problem. I removed f248a85 this mod2 to resolve the issue. as we had only single wide dataset now. (let me know if you have alternative suggestions)

[vedhav]

Thanks @kartikeyakirar LGTM now.

[m7pr]

@kartikeyakirar does this close #640 ?

[kartikeyakirar]

yes