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

Update gui.py #695

Closed
wants to merge 1 commit into from
Closed

Update gui.py #695

wants to merge 1 commit into from

Conversation

0fbcb238c0
Copy link

Update browserfilter to allow saving playlists as m3u8 by default.

Update browserfilter to allow saving as m3u8 by default.
@Et0h
Copy link
Contributor

Et0h commented Sep 28, 2024

Syncplay does not actually support .m3u8 because we do not support the metadata this format can contain. The only reason we still had loading for it is because we used to allow people to load/save in the .m3u8 name but without it supporting the standard and as part of #408 back in 2021 we removed the saving but kept the loading in case someone had previously saved the playlist. We entitled this as "Begin move from m3u/m3u8 to txt for playlist" as it was only intended to be a temporary measure.

Given that we have no plans to support loading true m3u8 files and it seems to be causing confusion it might be best to remove the load m3u8 option.

@0fbcb238c0
Copy link
Author

Syncplay does not actually support .m3u8 because we do not support the metadata this format can contain. The only reason we still had loading for it is because we used to allow people to load/save in the .m3u8 name but without it supporting the standard and as part of #408 back in 2021 we removed the saving but kept the loading in case someone had previously saved the playlist. We entitled this as "Begin move from m3u/m3u8 to txt for playlist" as it was only intended to be a temporary measure.

Given that we have no plans to support loading true m3u8 files and it seems to be causing confusion it might be best to remove the load m3u8 option.

Alright, good to know. I will close this issue and change the extension of my playlist to txt.
Thank you

@0fbcb238c0 0fbcb238c0 closed this Sep 30, 2024
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