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

Play advertisements #28

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

mediaminister
Copy link
Collaborator

This implements playing advertisements before programs.

@michaelarnauts
Copy link
Collaborator

Do you know why they need to know the weather? Are other ads provided then?

@mediaminister
Copy link
Collaborator Author

Yes, this is used for "Weather based marketing". And Facebook does this, so everyone else copies it.

@michaelarnauts
Copy link
Collaborator

Never knew this was a thing!

@dagwieers dagwieers added the enhancement New feature or request label May 22, 2020
@dagwieers
Copy link
Collaborator

I rebased this to v0.2.0.

@michaelarnauts
Copy link
Collaborator

@mediaminister what's the status of this PR? Can this be merged, or does this have issues?

I remember reading somewhere that the playlist approach causes some problems, but I'm not sure anymore where I read that or what issues it could cause.

@mediaminister
Copy link
Collaborator Author

Not mergeable, this PR breaks Kodi resume points...
I didn't found a solution yet.

@dagwieers dagwieers added this to the Future milestone Jan 30, 2021
@michaelarnauts michaelarnauts removed this from the Future milestone Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants