-
Notifications
You must be signed in to change notification settings - Fork 2
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
Updated to scorecards for 2025 site #724
Open
struan
wants to merge
56
commits into
master
Choose a base branch
from
2025-scorecards
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
makes it easier to update, plus easier to have staging site use different year
fix incorrect heading names fix some bad score data
If it's the current year then link to the url without the year prefix, otherwise add year prefix.
also get this to respect the currently selected year
- Added some responsiveness improvements for tablet and screens below 1200px. Council comparison won't be display below a large screen. - Added Previous year score checkbox, by default we will only display year variation, but this switch will allow to also display the previous year score for current council and any council we are comparing with.
Was running a query for every question to get the criteria and then one for every section to get the council so make sure those things are fetched in the initial query
To make this element more reusable I have added the class js-table-year-comparison so instead of tackling specifically tables like the council one it tackles tables with the class js-table-year-comparison
Rather than selecting it based on council and year we can just create a link on import which makes things easier
When we import the scores take a previous year argument and use that to populate the previous_year link
this is used in the automcomplete JSON data which is broken by new lines
Just makes it a bit more comprehensible
e.g for the first year's scorecards
Also make config per year
for labeling the most improved councils on import, and then for generating graphics etc Also add plan scores to admin interface
Does this for overall per council type and overall per section
Also put things in a per year folder
easier to update in the database rather than the settings file
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is mostly small changes to allow multiple years of actions scores.
Part of #645
Fixes #722
Fixes #723