Skip to content

Fix for current twitch api #3

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.

Sign up for GitHub

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

Open
robbiemu wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Fix for current twitch api #3

Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
4cc4de2
added clientid support
robbiemu Dec 14, 2016
7f217d6
changed clientid tag in send json
robbiemu Dec 14, 2016
2cdaac3
changed clientid tag in send json
robbiemu Dec 14, 2016
b2d89b9
moved Client-ID to headers
robbiemu Dec 14, 2016
2c51d33
moved Client-ID to headers
robbiemu Dec 14, 2016
d925843
added version
robbiemu Dec 14, 2016
5b3cfa3
debugging headers
robbiemu Dec 14, 2016
3ba7d71
moving from jsonp
robbiemu Dec 14, 2016
a7eec95
debugging return
robbiemu Dec 14, 2016
fe6a34a
debugging return
robbiemu Dec 14, 2016
d66f40a
debugging return
robbiemu Dec 14, 2016
70da245
rewrote ajax call
robbiemu Dec 14, 2016
d48b71c
added class change so we can watch DOMSubtreeModified for the images
robbiemu Dec 14, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 17 additions & 5 deletions twitch-channel-status.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,13 +3,21 @@

// Create a closure with a reference to our script
(function (document, $script) {
console.log('[twitch-channel-status] robbimu fork - v1.0.14')
// Allow customizing the script with various data-* attributes
var attribute = $script.attr("data-attribute") || "data-twitch-channel",
var clientid = $script.attr("data-clientid") || false
attribute = $script.attr("data-attribute") || "data-twitch-channel",
interval = parseInt($script.attr("data-interval")) || false,
onlineImage = $script.attr("data-online-image") || 'data:image/svg+xml,<?xml version="1.0"?><svg height="20px" viewBox="0 0 20 20" version="1.1" xmlns="http://www.w3.org/2000/svg"><circle cx="10" cy="10" r="8" fill="green"/></svg>',
offlineImage = $script.attr("data-offline-image") || 'data:image/svg+xml,<?xml version="1.0"?><svg height="20px" viewBox="0 0 20 20" version="1.1" xmlns="http://www.w3.org/2000/svg"><circle cx="10" cy="10" r="8" fill="red"/></svg>',
minInterval = 30; // seconds

// if there is no clientId we cannot function!
if(!clientid) {
console.log('[twitch-channel-status] `Client-ID` is now mandatory for kraken requests. (see: https://blog.twitch.tv/client-id-required-for-kraken-api-calls-afbb8e95f843#.cxn9jkpgi )')
console.log('[twitch-channel-status] your `Client-ID` can now be set as a `data-clientid` data attribute on the script tag')
}

document.refreshTwitchChannelStatuses = function () {
var channels = {};

Expand Down Expand Up @@ -37,14 +45,17 @@

// Ask twitch for the status of all channels at once
$.ajax({
url: "https://api.twitch.tv/kraken/streams",
data: {"channel": Object.keys(channels).join(","), "limit": Object.keys(channels).length},
cache: false,
dataType: "jsonp"
type: 'GET',
url: 'https://api.twitch.tv/kraken/streams',
data: `channel=${Object.keys(channels).join(",")}&limit=${Object.keys(channels).length}`,
headers: {
'Client-ID': clientid
}
}).done(function (data) {
// We can only handle 100 online channels at a time :(
if (data.streams.length < data._total) {
console.warn("refreshTwitchChannelStatuses couldn't load all online channels! Please reduce the number of channels you are trying to check.");
console.dir(data.streams)
}

// Build a hash of who's online for performance
Expand All @@ -66,6 +77,7 @@
var $img = imgs[i];
var src = $img.attr("data-" + (online ? "online" : "offline") + "-image") || image;
$img.attr("src", src);
online? $img.addClass('online'):$img.removeClass('online')
}
}

Expand Down