-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
base: master
Are you sure you want to change the base?
Conversation
94a80ce
to
d3d05f9
Compare
Do you know why they need to know the weather? Are other ads provided then? |
Yes, this is used for "Weather based marketing". And Facebook does this, so everyone else copies it. |
Never knew this was a thing! |
fb07177
to
a7bddf0
Compare
I rebased this to v0.2.0. |
@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. |
Not mergeable, this PR breaks Kodi resume points... |
This implements playing advertisements before programs.