Tom Jemmett

Synthetic NEWS vignette

1

Currently, in the synthetic news data vignette, the data is loaded [directly from a url](https://github.com/nhs-r-community/NHSRdatasets/blob/82304f9bc6750fe5d5d8347db687232cfbd92d68/vignettes/synthetic_news_data.Rmd#L52). This is a potential issue for rendering with CRAN: occasionally file downloads can fail, which...

GitHub pages always deploys latest version

3

The pkgdown action always deploys latest version of master branch, but this will often not reflect what is deployed to cran. Options: 1. we should only deploy when a new...

Contribution instructions should state dataset limitations (CRAN)

3

Currently the contribution section on Readme.md doesn't state any of the limitations imposed by CRAN, namely that the entire package must be

Developers should have a precommit hook to prevent commits to Readme.Rmd without a commit to Readme.md (`usethis::use_readme_rmd()` will create this). As the developer needs to set this up locally, we...

Should there be more palettes?

3

Currently we have one palette that uses a handful of the colours from the Identity (Blue, Green, Red, Purple, Orange), and then the levels as defined by the Identity (Blues,...

Create a Word document template that follows the NHS Identity guidelines

Create a html template that follows the NHS Identity guidelines

Create a LaTeX template that follows the NHS Identity guidelines

plotly hover tooltips need improving

5

current plotly implementation uses default tooltips, these could do with being improved

The ggplot object uses a formatter in scale_colour_manual ([labels = ptd_title_case](https://github.com/nhs-r-community/NHSRplotthedots/blob/cff53684f8f2fda64d2c23b35365d41303f93b81/R/ptd_create_ggplot.R#L154)) to convert text like "common_cause" to "Common Cause". It also hides the title "point_colour". While we base the plotly...