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

edited the logic a little bit #29

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

edited the logic a little bit #29

wants to merge 10 commits into from

Conversation

cm86
Copy link

@cm86 cm86 commented Jun 20, 2022

Hello,
this is my first time i'm writing public in python.
But i tried to edit your script a little bit.
Because of if you always delete and create the playlist new, the watch status is gone, i have modified the script.

  • outsourcing of playlist to file (mcuchronologicalplaylist.txt) --> make editing easier
  • if there is no playlist with the name you choosed --> it creates the playlist
  • if there is already a playlist with the same name --> it looks inside the playlist and gives you the choice
  • -> if you have more items in you library you can add them and check the order
  • -> if you have the wrong content in your playlist you can remove them and check order
  • -> if the playlist is up to date you can check the order

@cm86 cm86 marked this pull request as ready for review June 20, 2022 21:37
@PJnes
Copy link
Owner

PJnes commented Jun 26, 2022

Thanks a lot for this PR. There's quite a bit here, will need some time to review it!

cm86 added 2 commits July 1, 2022 13:26
Line 175 needs to be otherway round, because he didn't found that "Thor" was missing in my playlist... ;)
 --> the error wasn't here
it's in the search function wich delivers more then one match...

I'm on the line for fixing...
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