Skip to content

[add] ETTV rewriter#2645

Merged
liiight merged 5 commits intoFlexget:developfrom
flakyton:ettv-rewriter
Apr 2, 2020
Merged

[add] ETTV rewriter#2645
liiight merged 5 commits intoFlexget:developfrom
flakyton:ettv-rewriter

Conversation

@flakyton
Copy link
Copy Markdown
Contributor

@flakyton flakyton commented Apr 1, 2020

Motivation for changes:

Make ETTV play nice with html plugin

Detailed changes:

  • Implemented ETTV rewriter plugin

@flakyton flakyton changed the title ETTV rewriter [add] ETTV rewriter Apr 1, 2020
@paranoidi
Copy link
Copy Markdown
Member

Looks good to me. Thanks for contributing!

'www.ettvdl.com',
]

class UrlRewriteETTV:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

seems there's a missing line here. We style the code with black. You can install pre-commit and enable it for the repo and it will take care of this stuff

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fixed

raise UrlRewritingError(e)
tag_a = soup.find('a', attrs={'class': 'download_link'})
if not tag_a:
raise UrlRewritingError('Unable to locate download link from url %s' % url)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

please use fstrings

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fixed

@liiight liiight merged commit 8683a5e into Flexget:develop Apr 2, 2020
@liiight
Copy link
Copy Markdown
Member

liiight commented Apr 2, 2020

Thanks!

@flakyton flakyton deleted the ettv-rewriter branch April 3, 2020 03:56
StefftheEmperor2 pushed a commit to StefftheEmperor2/Flexget that referenced this pull request Apr 25, 2020
* [add] ETTV rewriter plugin

* Ignore VSCode settings

* Formatting

* Use f-strings

* Use select instead of find
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.

3 participants