-
Notifications
You must be signed in to change notification settings - Fork 12
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
Hotfix/impact numbers #853
Conversation
116f851
to
77150d2
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #853 +/- ##
=======================================
Coverage 77.73% 77.73%
=======================================
Files 28 28
Lines 566 566
=======================================
Hits 440 440
Misses 126 126 ☔ View full report in Codecov by Sentry. |
democracy_club/templates/home.html
Outdated
@@ -26,8 +26,7 @@ <h2>Our impact in numbers</h2> | |||
and local elections.</p> | |||
|
|||
<ul class="ds-switcher ds-showcase" style="--gridCellMin: 16ch; --gridGap: 1.5rem;"> | |||
<li><strong>219m</strong> reminders to vote sent via partnerships in 2019</li> | |||
<li><strong>3.5m</strong> people used our services in the last general election</li> | |||
<li><strong>14m</strong> people used our services in the 2024 general election</li> | |||
<li><strong>83%</strong> of users say that our services make them more likely to vote</li> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pmk01 Regarding "more likely to vote", with the addition of the "no change" option, this number has become more complicated to report. Do we want to revert to 'found useful'? If I average the numbers from WDIV and WCIVF, we are at 86%.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I agree, move back to 'found useful'.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The specific question is 'did you find what you were looking for?' Is there's a better way to summarise this?
"85% of users found the information they needed", or similar?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've made this change.
045f073
to
714f4a3
Compare
714f4a3
to
a101185
Compare
This PR updates the impact metrics following the 2024 General Election, includes a security patch for Django, and adjusts indentation.