You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We could do with a workflow script that ensures a public dashboard is meeting the necessary requirements (as much as we can reasonably regulate via a workflow). I'm thinking:
ensure the necessary cookie control elements are present
ensure an accessibility statement is present
ensure the support_links panel is present
In addition, it might be worth considering if certain parameters are filled in appropriately (although I suspect we'll have plenty of these covered just by mandating the above panels are present and their associated validation):
google_analytics_key
title
name of dashboard in cookies
...
Describe alternatives you've considered
Thought about a commit hook, but I like it the idea of something being added to the PR testing so that we've got clearer sight of what's going on.
Additional context
This follows on from the Attendance dashboard team removing the necessary cookie control code from their server.R script and breaking that functionality for 3 weeks.
The text was updated successfully, but these errors were encountered:
I like this, would be interesting to think about whether there's anything we do to automate monitoring as a GitHub organisation when this workflow is run and whether its last run on the default branch of a given repo passed or failed.
Describe the solution you'd like
We could do with a workflow script that ensures a public dashboard is meeting the necessary requirements (as much as we can reasonably regulate via a workflow). I'm thinking:
In addition, it might be worth considering if certain parameters are filled in appropriately (although I suspect we'll have plenty of these covered just by mandating the above panels are present and their associated validation):
Describe alternatives you've considered
Thought about a commit hook, but I like it the idea of something being added to the PR testing so that we've got clearer sight of what's going on.
Additional context
This follows on from the Attendance dashboard team removing the necessary cookie control code from their server.R script and breaking that functionality for 3 weeks.
The text was updated successfully, but these errors were encountered: