Support/wagtail 62 #1052
Support/wagtail 62 #1052
Conversation
nickmoreton
force-pushed
the
support/wagtail-62
branch
from
6644091
to
f4a00b3
Compare
Also update test_get_user_edit_form_with_default_form to use UserViewSet
There was a problem hiding this comment.
Issue: The customisation to the aging pages report (see AgingPagesReportView in rca/reports/views.py) appears to have stopped working - probably due to the shift to the Universal Listings UI.
Hi @alxbridge Please could you expand on the issue you have seen. It seems to be working OK for me locally. |
Ah it works OK with initial data, e.g. not pulled data. But the export isn't visible with remote data pulled in. I'll look into it. |
I was checking the wrong Report page (page types usage) the Aging pages report looks OK to me with pulled data. |
Sorry for not being more specific to start with: the customised report adds a column for the page's URL which doesn't show up for me |
I think that column is only in the exported data, I can see it OK in an export on the master branch and the upgrade branch. I'm not seeing that column on either branch in the list view. |
I've just re-tested and it does actually get output as expected for the full report: the problem only seems to arise if you apply filters before the export. |
I looked at this @alxbridge and you are correct. What I also discovered is if you refresh the page with the filter querystring in place it does work then. It's because the to_row_dict doesn't get called with filtering updates but does with page loading. |
Hi @alxbridge After a few wrong turns I finally fixed this. I wish there were more chages required but it was simple in the end 😄 1f6f8e9 |
No description provided.