Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

json-rpc: add new notification OnScanStarted for libraries #1487

Merged
merged 1 commit into from Oct 1, 2012

Conversation

Projects
None yet
4 participants
Contributor

alcoheca commented Sep 27, 2012

@Montellese is this acceptable? I need to this to avoid spamming upnp clients during library scans

Owner

Montellese commented Sep 27, 2012

Sure, I already figured that we'd need something like that sooner or later as during a library update (especially when there's a new source to be scanned) there can be a lot of updates and it might be easier for clients to just ignore all of them and then do a proper sync after OnScanFinished.

@Montellese Montellese was assigned Sep 27, 2012

Owner

MartijnKaijser commented Sep 27, 2012

+1 from me. Could be usefull for scripts

Contributor

amet commented Sep 27, 2012

as much as @jimfcarroll hates the monitor, maybe we should extend it to receive this as well if we need it in scripts/addons

@Montellese Montellese added a commit that referenced this pull request Oct 1, 2012

@Montellese Montellese Merge pull request #1487 from alcoheca/rpc-scanstart
json-rpc: add new notification OnScanStarted for libraries
ef7f03e

@Montellese Montellese merged commit ef7f03e into xbmc:master Oct 1, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment