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

Mediaklikk extractor #17453

Closed
wants to merge 5 commits into from
Closed

Mediaklikk extractor #17453

wants to merge 5 commits into from

Conversation

tmarki
Copy link

@tmarki tmarki commented Sep 6, 2018

Please follow the guide below

  • You will be asked some questions, please read them carefully and answer honestly
  • Put an x into all the boxes [ ] relevant to your pull request (like that [x])
  • Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

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:

  • 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
  • New extractor
  • New feature

Description of your pull request and other information

Extractor for Hungarian state media (MTVA) video repository hosted at https://www.mediaklikk.hu/mediatar/

@tmarki tmarki changed the title Mediaklikk Mediaklikk extractor Sep 6, 2018
youtube_dl/extractor/mediaklikk.py Outdated Show resolved Hide resolved
youtube_dl/extractor/mediaklikk.py Outdated Show resolved Hide resolved
youtube_dl/extractor/mediaklikk.py Outdated Show resolved Hide resolved
youtube_dl/extractor/mediaklikk.py Outdated Show resolved Hide resolved
youtube_dl/extractor/mediaklikk.py Outdated Show resolved Hide resolved
@tmarki
Copy link
Author

tmarki commented Sep 8, 2018

Please see commit with fixes.

@dstftw
Copy link
Collaborator

dstftw commented Oct 30, 2019

Also read coding conventions.

Copy link

@mrx23dot mrx23dot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works greats, ready to merge back

@mrx23dot mrx23dot mentioned this pull request May 2, 2020
9 tasks
@mrx23dot
Copy link

mrx23dot commented May 2, 2020

Finished work in #25098

@dstftw dstftw force-pushed the master branch 2 times, most recently from 5e26784 to da2069f Compare September 13, 2020 13:51
@VinhoXie
Copy link

Work well. You can also have a check at this article about mediaklikk downloader.

pukkandan pushed a commit to yt-dlp/yt-dlp that referenced this pull request Sep 6, 2021
nixxo pushed a commit to nixxo/yt-dlp that referenced this pull request Nov 22, 2021
@dirkf dirkf closed this Aug 1, 2023
@dirkf dirkf added the defunct PR source branch is not accessible label Oct 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
defunct PR source branch is not accessible pending-fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants