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

Add ability to upload theme music #658

Closed
wants to merge 6 commits into from
Closed

Add ability to upload theme music #658

wants to merge 6 commits into from

Conversation

ReenigneArcher
Copy link
Contributor

Added theme endpoints
Fixed comments in art endpoints.

Added theme endpoints
Fixed comments in art endpoints.
@Hellowlol
Copy link
Collaborator

Hellowlol commented Feb 6, 2021

This class supports many things Does everything has a theme?

@ReenigneArcher
Copy link
Contributor Author

This class supports many things Does everything has a theme?

According to this old documentation, only movie, tv show, and artist (not tv episode or tv season)

https://web.archive.org/web/20150113085312/http://dev.plexapp.com/docs/agents/models.html

Where should I put it to limit what items can have a theme?

@ReenigneArcher
Copy link
Contributor Author

Think I found where... videos.py

Remove themes from base and add to video.py and audio.py
Add ability to upload theme music to movies and tv shows
Add ability to upload theme songs to artists
@ReenigneArcher
Copy link
Contributor Author

Sorry, I don't understand why the checks are failing...

@JonnyWong16
Copy link
Collaborator

Themes should be moved to a mixin similar to #648.

@blacktwin
Copy link
Collaborator

This isn't something that we can do in the UI currently and the way you'd undo it is potentially destructive to the item. From my experience of playing this feature, you'd have to dance the item away to clear out the themes. I'd assume when danced, because the item is given a new ratingKey, all it's history is gone. It's why I didn't create a PR then and posted the feature request instead.

Maybe add a disclaimer of using the feature or guidance on what to do (dance) if you regret your decision. For Shows libraries, where there is a toggle for the library, you can turn off themes. For non Show libraries you can't. So unless you upload a silent audio file you're stuck with a theme on that item or you dance.

@JonnyWong16 JonnyWong16 marked this pull request as draft June 7, 2021 03:12
@ReenigneArcher ReenigneArcher mentioned this pull request Jan 25, 2022
8 tasks
@ReenigneArcher
Copy link
Contributor Author

Closing as #879 replaces this and I longer have access to the repo I made these changes in.

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.

4 participants