Merge lp://qastaging/~michael.nelson/ubuntu-webcatalog/1019326-missing-ratings-summary into lp://qastaging/ubuntu-webcatalog

Proposed by Michael Nelson
Status: Merged
Approved by: Michael Nelson
Approved revision: 182
Merged at revision: 174
Proposed branch: lp://qastaging/~michael.nelson/ubuntu-webcatalog/1019326-missing-ratings-summary
Merge into: lp://qastaging/ubuntu-webcatalog
Diff against target: 256 lines (+100/-25)
6 files modified
src/webcatalog/admin.py (+6/-0)
src/webcatalog/management/commands/import_all_ratings_stats.py (+13/-1)
src/webcatalog/management/commands/import_ratings_stats.py (+25/-13)
src/webcatalog/tasks.py (+7/-1)
src/webcatalog/tests/test_commands.py (+44/-9)
src/webcatalog/tests/test_views.py (+5/-1)
To merge this branch: bzr merge lp://qastaging/~michael.nelson/ubuntu-webcatalog/1019326-missing-ratings-summary
Reviewer Review Type Date Requested Status
Łukasz Czyżykowski (community) Approve
Review via email: mp+137619@code.qastaging.launchpad.net

Commit message

Add admin class for ReviewStatsImport and optional --download-all flag for import_ratings_stats.

Description of the change

After some investigation on bug 1015604, this branch adds:

1) An admin class for ReviewStatsImport, so we can view/update when the last import occurred,
2) An --download-all flag for the import_ratings_stats command which forces all stats for the series to be downloaded, rather than requesting the optimised last X days,
3) An extra task so that we can request all ratings stats to be downloaded without needing a webop.

`fab test`

To post a comment you must log in.
Revision history for this message
Łukasz Czyżykowski (lukasz-czyzykowski) wrote :

All great

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches