Removing the performance banner that we had up this morning during the initial loading issues.
There was an issue in the industry by subject drop down inputs where the filtering of the SIC codes by industry was being overwritten by the switching off of SIC codes when industry was selected.
The app now has two observeEvent() functions that look for changes to either the breakdown or industry area inputs and then update the SIC dropdown accordingly, so you should be able to filter the SIC dropdown using the industry area dropdown, and this should continue working after switching the breakdown to industry and back to something else.
Pull request checklist
Please check if your PR fulfils the following:
[ No ] Tests for the changes have been added (for bug fixes / features)
[ No ] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ Yes ] Tests have been run locally and are passing (run_tests_locally())
[ Yes ] Code is styled according to tidyverse styling (checked locally with tidy_code())
Pull request overview
Removing the performance banner that we had up this morning during the initial loading issues.
There was an issue in the industry by subject drop down inputs where the filtering of the SIC codes by industry was being overwritten by the switching off of SIC codes when industry was selected.
The app now has two observeEvent() functions that look for changes to either the breakdown or industry area inputs and then update the SIC dropdown accordingly, so you should be able to filter the SIC dropdown using the industry area dropdown, and this should continue working after switching the breakdown to industry and back to something else.
Pull request checklist
Please check if your PR fulfils the following:
run_tests_locally()
)tidy_code()
)