Skip to content
This repository was archived by the owner on Nov 27, 2024. It is now read-only.

Conversation

@wwark
Copy link

@wwark wwark commented Mar 7, 2019

  • Test Dailymotion OK
  • Test Youtube OK

Do you think this change can have some strange behavior ?

Thank you,
Wwark

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.2%) to 98.692% when pulling a5477b2 on wwark:master into cb98ddc on willforde:master.

@willforde
Copy link
Owner

Sorry for the delay. That fix should work ok, but i'm not sure.
So I have fixed it in a different way, ware it sets listitem.setContentLookup(False) only when a dailymotion video is detected. Thanks for the initial solution anyway.
If you have any issues with it, let me know.

See. dafcdeb

@wwark
Copy link
Author

wwark commented Mar 12, 2019

Thanks a lot @willforde !

For information the version number is not ok in the file "history.md"
I close this PR.

Wwark.

@wwark wwark closed this Mar 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants