Rob--W / stackexchange-notifications

Realtime desktop notifications for Stack Exchange sites.
44 stars 8 forks source link

Chrome version broken with sync changes #16

Closed kav2k closed 9 years ago

kav2k commented 9 years ago

Looks like #11 broke the Chrome version.

This line from migration code fails because storage is undefined, grinding the extension to a halt.

Mea culpa for not finding time to get an API key to test #11, I suppose.

kav2k commented 9 years ago

Funny how I did not notice it for a full month, nor any of your other users.

Rob--W commented 9 years ago

Patches are welcome ;)

If you don't have time to submit a PR, I'll commit a patch tomorrow.

PS. I didn't notice, because I'm using the Firefox addon instead of the Chrome extension.