search

phaistonian / MyExtensions

Are you an extensions developer? Keep track of your Chrome extensions. Be notified for new comments, ratings and more!
https://chrome.google.com/webstore/detail/igejgfmbjjjjplnnlgnbejpkpdajkblm
52 stars 24 forks source link

Chrome Web Store Update #21

Closed neocotic closed 12 years ago

neocotic commented 12 years ago

Yet another fix for a Web Store update. They added a new required parameter to the webstore/ajax/detail endpoint that seems to track states, as well as embedding the JSON response in another array. Finally, they changed the image URLs to be absolute (i.e. now include the https: protocol).

I've added fixes to handle all of these changes and it works fine for me.

These issues were discovered while investigating #20.

neocotic commented 12 years ago

Found a slight problem with this when trying on a different machine. It's something to do with the new required parameter I think. I'll have a look at this now, but might have to be more creative.

neocotic commented 12 years ago

OK, I've fixed the problem where my previous fix wouldn't work for authenticated (logged in) users. I've created a workaround and it now works perfectly. I've also made a few improvements to parsing Google's dirty JSON/JavaScript as well as simplifying the pv parameter extraction even further.

Please can these changes be merged ASAP as the extension is currently broken without them.