Skip to content

#46 Add a csv download for reports - #48

Merged
pposiada merged 11 commits into
mainfrom
46-add-download-xlsx-functionality
Jun 29, 2026
Merged

#46 Add a csv download for reports#48
pposiada merged 11 commits into
mainfrom
46-add-download-xlsx-functionality

Conversation

@pposiada

Copy link
Copy Markdown
Collaborator

No description provided.

@pposiada pposiada linked an issue Jun 10, 2026 that may be closed by this pull request
@github-actions

github-actions Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

badge

Code Coverage Summary

Filename                                     Stmts    Miss  Cover    Missing
-----------------------------------------  -------  ------  -------  ---------
R/BlockConditions.R                             21      21  0.00%    23-61
R/create_rel_risk_transformator.R               42      42  0.00%    41-87
R/forest_plot_decorator.R                      164     164  0.00%    97-373
R/ggplot_decorator.R                           245     245  0.00%    105-376
R/merge_levels_transformator.R                 146     146  0.00%    11-240
R/or_filtering_transformator_view_model.R       27      27  0.00%    25-61
R/or_filtering_transformator.R                 282     282  0.00%    69-397
R/patchwork_plot_decorator.R                    46      46  0.00%    18-68
R/r_access_utilities.R                          46      46  0.00%    32-152
R/ReportManager.R                              401     401  0.00%    41-743
R/title_footer_decorator.R                     130     130  0.00%    105-242
R/tm_report_manager.R                          719     719  0.00%    31-896
R/watermark_decorator.R                         61      61  0.00%    23-87
TOTAL                                         2330    2330  0.00%

Diff against main

Filename                 Stmts    Miss  Cover
---------------------  -------  ------  --------
R/ReportManager.R          +94     +94  +100.00%
R/tm_report_manager.R      +23     +23  +100.00%
TOTAL                     +117    +117  +100.00%

Results for commit: e9f7603

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@pposiada
pposiada requested a review from kpagacz June 15, 2026 10:35
Comment thread R/tm_report_manager.R
a.download = msg.filename;
document.body.appendChild(a);
a.click();
setTimeout(function() { document.body.removeChild(a); URL.revokeObjectURL(url); }, 100);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shall we use a normal shiny file downloader?

@pposiada
pposiada requested a review from kpagacz June 26, 2026 09:36
@pposiada
pposiada merged commit f2c00ea into main Jun 29, 2026
@pposiada
pposiada deleted the 46-add-download-xlsx-functionality branch June 29, 2026 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add "Download xlsx" functionality to the Report Manager module

2 participants