reading-am / reading

The dreams! The dreams! It's all just absurdity in the light of day but the dreams!
MIT License
10 stars 0 forks source link

local extension config #349

Open sjkaliski opened 12 years ago

sjkaliski commented 12 years ago

@leppert was wondering what your local config looked like for the chrome extension. i'm wrapping up some work on the back end for notifications and wanted to start integrating with the extension. i tried to add an unpackaged extension (i set up the shared folder, etc) but ran into manifest versioning issues (it wasn't including int he manifest). tried to do some adjusting, but figured you had a better sense of what might be going on.

leppert commented 12 years ago

@sjkaliski I have mine set up as an unpacked extension like you mentioned:

Chrome did give me a warning the last time I installed it, which is where https://github.com/leppert/reading/issues/310 originated, but it still allowed me to install. Where are you hitting a wall?

sjkaliski commented 12 years ago

This is the error chrome is returning:

Could not load extension from '/Users/Stephen/Documents/Work/reading/reading-extensions/chrome'. The 'manifest_version' key must be present and set to 2 (without quotes). See developer.chrome.com/extensions/manifestVersion.html for details.

leppert commented 12 years ago

Maybe they've upgraded Chrome such that you can no longer install new manifest v1 extensions. Currently working on upgrading it.

sjkaliski commented 12 years ago

Let me try an older version of chrome, or just work on firefox for a bit. Keep me posted on that

leppert commented 12 years ago

@sjkaliski the Chrome extension repo has been updated to manifest v2: https://github.com/leppert/reading-extensions/commit/4b8b765b1123660fe0b874ec16806d51624db25f. Would you mind kicking the tires, making sure it installs, before I push out to the Chrome Store?

leppert commented 12 years ago

@sjkaliski also, note the instructions for using the extension locally: https://github.com/leppert/reading-extensions#using-locally

sjkaliski commented 12 years ago

@leppert i'll take it for a spin now

sjkaliski commented 12 years ago

@leppert works good for me