This adds all code logic for the All LA level page.
Pull request checklist
Please check if your PR fulfills the following:
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[x] Tests have been run locally and are passing (shinytest2::test_app())
[x] Code is styled according to tidyverse styling (checked locally with styler::style_dir() and lintr::lint_dir())
What is the current behaviour?
The All LA level page and code is not currently in the app.
What is the new behaviour?
This provides all logic for the All LA level page, from the simple static workshop script, to the development app to the shiny module code.
In this pull request, the download data functionality is also addressed. Applied to the All LA page and the LA level page (across tables and charts - .csv / .xlsx and .png / .html respectively). This is built using a module stored in the mod_app_helpers.R script.
There is also a major refactoring of the formatting of the {reactable} table columns. An issue was arising that due to applying the pretty_num() prior to rendering the table, numbers were actually string variables and so created weird sorting orders. Now the conversion to pretty_num() is done during rendering of the table and so sorting works as would be expected.
Anything else
The download functionality needs to be applied to the Region and Statistical Neighbour pages.
Pull request overview
This adds all code logic for the All LA level page.
Pull request checklist
Please check if your PR fulfills the following:
shinytest2::test_app())styler::style_dir()andlintr::lint_dir())What is the current behaviour?
The All LA level page and code is not currently in the app.
What is the new behaviour?
This provides all logic for the All LA level page, from the simple static workshop script, to the development app to the shiny module code.
In this pull request, the download data functionality is also addressed. Applied to the All LA page and the LA level page (across tables and charts -
.csv/.xlsxand.png/.htmlrespectively). This is built using a module stored in themod_app_helpers.Rscript.There is also a major refactoring of the formatting of the
{reactable}table columns. An issue was arising that due to applying thepretty_num()prior to rendering the table, numbers were actually string variables and so created weird sorting orders. Now the conversion topretty_num()is done during rendering of the table and so sorting works as would be expected.Anything else
The download functionality needs to be applied to the Region and Statistical Neighbour pages.