Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 51 commits into from
Jun 14, 2021

Conversation

martinctc
Copy link
Member

@martinctc martinctc commented May 13, 2021

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 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.

1 participant