youtube-dl icon indicating copy to clipboard operation
youtube-dl copied to clipboard

[SwitchTube] Add new extractor

Open belamenso opened this issue 2 years ago • 7 comments

Before submitting a pull request make sure you have:

  • [x] Searched the bugtracker for similar pull requests
  • [x] Read adding new extractor tutorial
  • [ ] Read youtube-dl coding conventions and adjusted the code to meet them Mostly, some lines are often longer than 80 columns, and no data point is extracted from more than a single source.
  • [x] Covered the code with tests (note that PRs without tests will be REJECTED)
  • [x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

  • [x] I am the original author of this code and I am willing to release it under Unlicense
  • [ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

  • [ ] Bug fix
  • [ ] Improvement
  • [x] New extractor
  • [ ] New feature

Description of your pull request and other information

This pull request adds tube.switch.ch, probably the largest Swiss website for universities to share lecture videos.

belamenso avatar Jun 10 '22 13:06 belamenso

I updated the code either exactly as suggested or similarly.

belamenso avatar Jun 18 '22 15:06 belamenso

I believe I was missing from __future__ import unicode_literals. I added it.

belamenso avatar Jun 19 '22 19:06 belamenso

Can someone please approve the CI tests? @dirkf

belamenso avatar Jun 20 '22 09:06 belamenso

I implemented the suggestions.

belamenso avatar Jun 22 '22 18:06 belamenso

Can somebody look into this?

belamenso avatar Jul 01 '22 08:07 belamenso

SwitchTube has apparently changes the playlist website format, making a large part of this extractor useless.

PLEASE DO NOT MERGE.

I will try to fix it in a week.

belamenso avatar Jul 05 '22 18:07 belamenso

I've split the previous SwitchTubeProfileOrChannelIE into SwitchTubeProfileIE (which was not changed on the website) and SwitchTubeChannelIE (which changed). It should work again now.

belamenso avatar Jul 31 '22 18:07 belamenso