-
-
Notifications
You must be signed in to change notification settings - Fork 235
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
New version of /reports main page. #1753
Conversation
85a1e6d
to
93e973a
Compare
93e973a
to
ae128a2
Compare
This will need redoing if #1753 is merged first.
This will need redoing if #1753 is merged first.
This is a much broader summary page, plus a body name autocomplete. This was originally implemented for fixmystreet.com in 8a6a4cc. It also adds '(no longer exists)' in the autocomplete next to bodies not covering any areas. You can supply the `--table` argument to `update-all-reports` to generate the old-style data.
ae128a2
to
f604fb2
Compare
Codecov Report
@@ Coverage Diff @@
## master #1753 +/- ##
==========================================
+ Coverage 71.64% 72.03% +0.39%
==========================================
Files 153 153
Lines 9847 9828 -19
Branches 1783 1782 -1
==========================================
+ Hits 7055 7080 +25
+ Misses 2126 2078 -48
- Partials 666 670 +4
Continue to review full report at Codecov.
|
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.
(if you copy the old template into your cobrand, it should continue to work fine).
CMC has done this, and it does!
Good idea to include a flag for backwards compatibility 🎉 As this is a breaking change for anyone who's overridden the reports/index.html
template (any idea of who's done this?) we should highlight it in the 2.1 release notes.
👍
This is a much broader summary page, plus a body name autocomplete.
This was originally implemented for fixmystreet.com in 8a6a4cc.
(It also adds '(no longer exists)' in the autocomplete next to bodies not covering any areas.)
You can supply the
--table
argument toupdate-all-reports
to generate the old-style data (if you copy the old template into your cobrand, it should continue to work fine).