ShelbyTV / shelby_gt

Rails API services for Shelby GT
2 stars 0 forks source link

C API needs to support JSONP #163

Closed spinosa closed 11 years ago

spinosa commented 11 years ago

From https://github.com/ShelbyTV/shelby-gt-web/issues/627 via @matyus

Internet Explorer 9

The Good: The layouts seem to be rendering pretty well, hurray for standards compliance!

The Bad:

LOG: error checking fb credentials 
SCRIPT1004: Expected ';' 
dashboard?fast=1&callback=jQuery17207841005673771346_1348930803673&include_children=true&limit=10&_=1348930806495, line 1 character 10
SCRIPT1004: Expected ';' 
following?callback=jQuery17207841005673771346_1348930803674&_=1348930806496, line 1 character 10
SCRIPT1004: Expected ';' 
frames?fast=1&callback=jQuery17207841005673771346_1348930803677&include_children=true&limit=10&_=1348930850059, line 1 character 10
SCRIPT1004: Expected ';' 
following?callback=jQuery17207841005673771346_1348930803679&frames=true&_=1348930880233, line 1 character 10
SCRIPT1004: Expected ';' 
dashboard?fast=1&callback=jQuery17207841005673771346_1348930803682&include_children=true&limit=10&_=1348930923165, line 1 character 10
LOG: unhandled urltools 
SCRIPT1004: Expected ';' 
dashboard?fast=1&callback=jQuery17207841005673771346_1348930803684&include_children=true&limit=10&_=1348930958692, line 1 character 10

(Assigning @iceberg901 though unsure who should really take this on)

<<<

assigning to @markerrj since he can implement this quickest before launch

markerrj commented 11 years ago

I believe this checkin adds proper JSONP support for the C Metal APIs...

https://github.com/ShelbyTV/shelby_gt/commit/d90e800f5801c41f85e0f1551b7dce0464810654

I don't have a VM with IE9 readily available to test. Will coordinate with @matyus.