Skip to content

Feature: generate a HTML report directly from an RMarkdown template#141

Merged
martinctc merged 51 commits into
mainfrom
feature/new-report-architecture
Jun 14, 2021
Merged

Feature: generate a HTML report directly from an RMarkdown template#141
martinctc merged 51 commits into
mainfrom
feature/new-report-architecture

Conversation

@martinctc

@martinctc martinctc commented May 13, 2021

Copy link
Copy Markdown
Member

Summary

This branch adds a generate_report2() function which enables generating HTML reports directly from an existing RMarkdown document / template. This is different from generate_report(), which instead creates a RMarkdown document from scratch by writing individual code chunks together.

Changes

The changes made in this PR are:

  1. Added generate_report2()
  2. Added a minimal report in the inst folders for testing purposes.
  3. Added a new wellbeing_report() as the first working report using the new generate_report2() implementation.
  4. Added create_tracking which produces a line chart trend visual that shows the average and the rolling 4 week average for a given metric. This is used within wellbeing_report() itself.

Checks

  • All R CMD checks pass
  • roxygen2::roxygenise() has been run prior to merging to ensure that .Rd and NAMESPACE files are up to date.
  • NEWS.md has been updated.

Notes

This fixes #<issue_number>

<other things, such as how to incorporate new changes>

Add function which enables generating HTML reports directly from an existing RMarkdown document / template.
@martinctc martinctc added the enhancement New feature or request label May 13, 2021
@martinctc martinctc requested a review from moralec May 13, 2021 10:33
@martinctc martinctc self-assigned this May 13, 2021
martinctc added 26 commits May 13, 2021 12:58
Necessary for report to run smoothly if a variable does not exist
Handle situations where 'replacement' is matched in the column names. And added more helpful comments
'Frequency of weekend work' visual now has reversed x-axis
And minor bug fixes including errors which emerge when no data is available
1. TOC numbering go 1-8
2. Remove 'Balance - ' from first page
3. Remove 'sample' in 'sample glossary'
@martinctc martinctc marked this pull request as ready for review June 14, 2021 13:38
@martinctc martinctc merged commit d36dba7 into main Jun 14, 2021
@martinctc martinctc deleted the feature/new-report-architecture branch June 14, 2021 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants