Skip to content
This repository has been archived by the owner on Oct 21, 2024. It is now read-only.

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!

@mediaminister mediaminister force-pushed the ads branch 4 times, most recently from fb07177 to a7bddf0 Compare May 11, 2020 17:05
@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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants