Skip to content
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

[pvr] Added Icon, thumbnail and fanart support for recordings #1455

Merged
merged 3 commits into from
Oct 3, 2012

Conversation

fetzerch
Copy link
Member

This PR adds support for showing icon, thumbnail and fanart images for recordings.
The paths to those images have to be provided by the addon.

Cheers,
Christian

@opdenkamp
Copy link
Member

could you stick the cosmetics in a separate commit please, so i can review it properly. thanks

@ghost ghost assigned opdenkamp Sep 26, 2012
@fetzerch
Copy link
Member Author

Sure, done. Due to the change in PVR_RECORDING, addons need to be recompiled. I guess that means we need to bump the version?

@opdenkamp
Copy link
Member

correct, both numbers need to be updated for this

@opdenkamp
Copy link
Member

rest is fine, if you could stick the skin fix in a seperate PR, then it'll be merged in next round

@fetzerch
Copy link
Member Author

Moved the skin change to a separate PR. However it's somewhat related. Without that the fanarts aren't shown.

@opdenkamp
Copy link
Member

no idea why github is still showing a green button, but the version number should be bumped to 1.4.0 for this now. could you update it please.

@fetzerch
Copy link
Member Author

fetzerch commented Oct 1, 2012

done

opdenkamp pushed a commit that referenced this pull request Oct 3, 2012
[pvr] Added Icon, thumbnail and fanart support for recordings
@opdenkamp opdenkamp merged commit d875510 into xbmc:master Oct 3, 2012
tru pushed a commit to plexinc/plex-home-theater-public that referenced this pull request Dec 15, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants