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

Jspf cleanup #2872

Merged
merged 13 commits into from
May 29, 2024
Merged

Jspf cleanup #2872

merged 13 commits into from
May 29, 2024

Conversation

mayhem
Copy link
Member

@mayhem mayhem commented May 15, 2024

This PR fixes some problems with JSPF serialization:

  1. The created_for element should be part of the extension.
  2. track identifier should be a list not a string.

This PR corrects the problems above, but still allows playlists in the old format to be submitted. All code fragments for this have been marked with a date, allowing us to clean this code up in a year.

This PR must be released WITH the troi PR of the same branch name!

TODO: Update the Troi release before merging.

@pep8speaks
Copy link

pep8speaks commented May 15, 2024

Hello @mayhem! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2024-05-28 19:36:12 UTC

@mayhem mayhem requested a review from amCap1712 May 15, 2024 14:24
@mayhem
Copy link
Member Author

mayhem commented May 28, 2024

Sorry for this sloppyness. Can you tell my heart wasn't into this?

@amCap1712
Copy link
Member

Sorry for the initial confusion. fixed now.

@amCap1712 amCap1712 merged commit 106119b into master May 29, 2024
3 checks passed
@amCap1712 amCap1712 deleted the jspf-cleanup branch May 29, 2024 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants