spree-ember icon indicating copy to clipboard operation
spree-ember copied to clipboard

Installation command

Open givanse opened this issue 9 years ago • 16 comments

This wouldn't work:

ember install spree-ember-storefront

I had to use a version number:

ember install [email protected]

Maybe document it? I'm not sure at what level this should be handled.

givanse avatar Jul 09 '15 22:07 givanse

Works for me with ember cli 0.2.7 - what CLI version are you on?

hhff avatar Jul 10 '15 11:07 hhff

I don't know why I didn't mention it...

"ember-cli": "1.13.1"

givanse avatar Jul 10 '15 11:07 givanse

kk - haven't had a chance to use 1.13.1 yet. I'll look into it eventually! currently working on Instance Initializers for the project

hhff avatar Jul 10 '15 11:07 hhff

I could not get it running at all

ember install spree-ember-storefront
version: 0.1.11
The `install` command does not take any arguments. You must use `install:npm` or `install:bower` to install a specific package.

so I defaulted to using:

npm install spree-ember-storefront

The behavior is different that described in the guide. And my ember app is not working as prescribed. Any leads?

raybesiga avatar Sep 22 '15 10:09 raybesiga

you may need to do ember install:addon spree-ember-storefront depending on your version of ember CLI

doing npm install spree-ember-storefront then ember generate spree-ember-storefront

is equivalent, however

hhff avatar Sep 22 '15 13:09 hhff

Hey Hugh,

So the issue was with my version of ember-cli. I cleared the cache, re-installed everything and the command worked fine. Now watching your talk at Ember London. Quite refreshing, though when I click on the products button at the bottom of the landing page, products page does not load though it seems to work fine on the command line..

Some sort of content security policy violation:

Content Security Policy violation: {"csp-report":{"document-uri":"http://localhost:4200/products","referrer":"","violated-directive":"style-src 'self'","effective-directive":"style-src","original-policy":"default-src 'none'; script-src 'self' 'unsafe-eval' localhost:49152 0.0.0.0:49152; font-src 'self'; connect-src 'self' ws://localhost:49152 ws://0.0.0.0:49152 http://0.0.0.0:4200/csp-report; img-src 'self'; style-src 'self'; media-src 'self'; report-uri http://0.0.0.0:4200/csp-report;","blocked-uri":"","source-file":"chrome-extension://gppongmhjkpfnbhagpmjfkannfbllamg","line-number":34,"column-number":15,"status-code":200}}
Content Security Policy violation: {"csp-report":{"document-uri":"http://localhost:4200/products","referrer":"","violated-directive":"style-src 'self'","effective-directive":"style-src","original-policy":"default-src 'none'; script-src 'self' 'unsafe-eval' localhost:49152 0.0.0.0:49152; font-src 'self'; connect-src 'self' ws://localhost:49152 ws://0.0.0.0:49152 http://0.0.0.0:4200/csp-report; img-src 'self'; style-src 'self'; media-src 'self'; report-uri http://0.0.0.0:4200/csp-report;","blocked-uri":"","source-file":"chrome-extension://noojglkidnpfjbincgijbaiedldjfbhh","line-number":50,"column-number":10,"status-code":200}}

Have you encountered that?

raybesiga avatar Sep 22 '15 13:09 raybesiga

you need to downgrade to these versions of Ember & Ember Data:

"ember": "1.12.0",
"ember-data": "1.0.0-beta.18"

We don't currently support the latest versions - we're slowly working on it but we really need a champion for it!

hhff avatar Sep 22 '15 14:09 hhff

You just update these in your bower.json and run a bower install :+1:

hhff avatar Sep 22 '15 14:09 hhff

My bower.json currently looks like this:

{
  "name": "madagascar",
  "dependencies": {
    "ember": "1.13.7",
    "ember-cli-shims": "ember-cli/ember-cli-shims#0.0.3",
    "ember-cli-test-loader": "ember-cli-test-loader#0.1.3",
    "ember-data": "1.13.8",
    "ember-load-initializers": "ember-cli/ember-load-initializers#0.1.5",
    "ember-qunit": "0.4.9",
    "ember-qunit-notifications": "0.0.7",
    "ember-resolver": "~0.1.18",
    "jquery": "^1.11.3",
    "loader.js": "ember-cli/loader.js#3.2.1",
    "qunit": "~1.18.0"
  }
}

Won't the changes affect the shims and test loader? Sorry, I'm a newbie to Ember so my question may be a little silly..

raybesiga avatar Sep 22 '15 14:09 raybesiga

give it a go and find out - this is the bower.json we developed Spree Ember against: https://github.com/ember-cli/ember-cli/blob/f924fb5d117473d881b0be6759807c1416be2be0/blueprints/app/files/bower.json

hhff avatar Sep 22 '15 14:09 hhff

You are a god! @hhff it is working perfectly. Now contemplating whether to go learn the internal workings of Ember, or figure it out as i go. And and advise regarding swapping out Foundation for my own custom design stylesheets and javascript? Or should I stick with it and until I really get how everything works?

raybesiga avatar Sep 22 '15 14:09 raybesiga

Ray, Don't mix any old random js with ember. Not recommended practice. Use Ember.

Hugh, what exactly needs to happen to get this updated? Trying to figure out if I could help?

Best, Kevin

On Tue, Sep 22, 2015 at 10:34 AM, Ray Besiga [email protected] wrote:

You are a god! @hhff https://github.com/hhff it is working perfectly. Now contemplating whether to go learn the internal workings of Ember, or figure it out as i go. And and advise regarding swapping out Foundation for my own custom design stylesheets and javascript? Or should I stick with it and until I really get how everything works?

— Reply to this email directly or view it on GitHub https://github.com/hhff/spree-ember/issues/88#issuecomment-142306464.

baraka2000 avatar Sep 22 '15 14:09 baraka2000

Thanks for the advice @baraka2000 and cheers to @hhff for getting me on this Ember train. Best regards from Kampala!

raybesiga avatar Sep 22 '15 14:09 raybesiga

@raybesiga - you can absolutely swap out Foundation if you want! We used Foundation because it was the most "plug and play" of the CSS Frameworks. It's meant to be swappable.

What i will say - is that Spree Ember is probably the biggest OSS Ember Addon project out there. Ember has got a really steep learning curve - so working with Spree Ember will be pretty tricky.

@baraka2000 - we basically need to get this lib working with the latest versions of Ember, Ember Data and Ember CLI. There's a failing PR up for it at the moment - but most of it is done there!

hhff avatar Sep 22 '15 14:09 hhff

6 months later, I finally had time to have a play with this again and thanks to your help here @hhff, I got this working perfectly. Amazing! So perhaps once I dig my heels in I might level up enough to actually help on this project. I certainly would like to see this grow, thanks Hugh!

For folks following the guides with newer ember-cli versions, the trick is specifying the correct older version of ember and ember-data as outlined by hhff above (in the bower.json file). Everything else worked for me.

baraka2000 avatar Jan 06 '16 05:01 baraka2000

Nice Kevin! On Wed, 6 Jan 2016 at 12:23 AM, Kevin Kornemann [email protected] wrote:

6 months later, I finally had time to have a play with this again and thanks to your help here @hhff https://github.com/hhff, I got this working perfectly. Amazing! So perhaps once I dig my heels in I might level up enough to actually help on this project. I certainly would like to see this grow, thanks Hugh!

For folks following the guides with newer ember-cli versions, the trick is specifying the correct older version of ember and ember-data as outlined by hhff above (in the bower.json file). Everything else worked for me.

— Reply to this email directly or view it on GitHub https://github.com/hhff/spree-ember/issues/88#issuecomment-169228860.

hhff avatar Jan 06 '16 14:01 hhff