Add screenshot support for for-purchase apps
Bug #821652 reported by
Anthony Lenton
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Ubuntu Apps Directory |
Fix Released
|
Low
|
Anthony Lenton | ||
Bug Description
Webcatalog currently relies on http://
For-purchase apps have screenshots served from software-
This would allow us to eventually customize the screenshot url for all apps, if needed, for free.
Related branches
lp:~elachuni/ubuntu-webcatalog/display-custom-screenshot
- Michael Nelson (community): Approve
-
Diff: 170 lines (+53/-14)8 files modifiedsrc/webcatalog/management/commands/import_ratings_stats.py (+4/-2)
src/webcatalog/models/applications.py (+1/-1)
src/webcatalog/static/css/webcatalog.css (+6/-0)
src/webcatalog/templates/webcatalog/application_detail.html (+4/-4)
src/webcatalog/tests/factory.py (+2/-2)
src/webcatalog/tests/test_commands.py (+14/-0)
src/webcatalog/tests/test_forms.py (+10/-0)
src/webcatalog/tests/test_views.py (+12/-5)
| tags: | added: kb-improvement sp-1 |
Revision history for this message
| Anthony Lenton (elachuni) wrote | #1 |
| Changed in ubuntu-webcatalog: | |
| status: | New → In Progress |
| importance: | Undecided → Low |
| assignee: | nobody → Anthony Lenton (elachuni) |
| Changed in ubuntu-webcatalog: | |
| status: | In Progress → Fix Committed |
| Changed in ubuntu-webcatalog: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Looking at the code the screenshot url was already being stored, so it was just a matter of displaying the non-standard screenshot if one was provided. Fix proposed in the related branch.