Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1.13 #89

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

1.13 #89

wants to merge 10 commits into from

Conversation

givanse
Copy link

@givanse givanse commented Jul 13, 2015

I'm still working on changes for the other addons, but I figured that it would be good to start pushing as I have them ready so I can have early feedback.

This PR upgrades the addons to ember-cli 1.13.1, but keeps ember 0.12.0 and ember-data 1.0.0-beta.18. The changes pave the way towards 2.0.

core

default: PASS
ember-1.13.0: PASS
ember-ember-data-1.13.0: FAIL
ember-release: PASS
ember-beta: FAIL
ember-canary: FAIL

storefront

default: PASS
ember-1.13.0: PASS
ember-ember-data-1.13.0: FAIL
ember-release: PASS
ember-beta: FAIL
ember-canary: FAIL

auth

default: PASS
ember-1.13.0: FAIL
ember-ember-data-1.13.0: FAIL
ember-release: FAIL
ember-beta: FAIL
ember-canary: FAIL

checkouts

default: PASS
ember-1.13.0: PASS
ember-ember-data-1.13.0: FAIL
ember-release: PASS
ember-beta: FAIL
ember-canary: FAIL

@givanse
Copy link
Author

givanse commented Jul 13, 2015

Will ember-1.11.0 be supported? I don't see a strong reason for that.
According to npm spree-ember 0.0.1-beta.1 was released almost at the same time of the release of ember v1.12.0, so nobody was using it with 1.11. I'll delete those configs.

@hhff
Copy link
Owner

hhff commented Jul 13, 2015

Yo @givanse you're the fucking MAN

Comments coming shortly

var SpreeService = container.lookup('service:spree');
CurrentOrderSupport.apply(SpreeService);

application.deferReadiness();
SpreeService._restoreCurrentOrder().finally(function() {
//TODO:https://github.com/emberjs/ember.js/issues/11247
application.advanceReadiness();
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have a work around for this coming soon

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👌

@givanse
Copy link
Author

givanse commented Jul 13, 2015

Only auth has failing tests for 1.13.0. It eludes me at the moment and right now I need to get other stuff done :/

Is the upgrade to ember-simple-auth 0.8.0 done? or half way there? Just noticed the version number in package.json, but there is also an open issue about it.

@hhff
Copy link
Owner

hhff commented Jul 13, 2015

I'll have a look at Auth!

Just haven't got around to bumping the version of simple auth yet - it was in beta when 0.0.1 was released

@hhff
Copy link
Owner

hhff commented Sep 1, 2015

@givanse - I should have time to work on this this week - can I have push for your fork? I'll do all my work on this branch

@givanse
Copy link
Author

givanse commented Sep 1, 2015

@hhff Yes, added you.

@hhff
Copy link
Owner

hhff commented Sep 1, 2015

thanks @givanse !

@givanse
Copy link
Author

givanse commented Sep 26, 2015

@hhff Did you update the backend that is used when the tests are run? (http://testing.spree-ember.com)

@hhff
Copy link
Owner

hhff commented Sep 28, 2015

@givanse - I haven't - it's been the same for a while. Does it need a change?

@givanse
Copy link
Author

givanse commented Sep 28, 2015

Some tests that were ok before are failing now. Simple things like:

was expecting 'Bags'
actual result 'Mugs'

Maybe a sorting change in the taxons? I fixed (edited for Mugs) and four more errors appeared. Maybe the seed data was edited?

I didn't spend much time looking into this, decided to wait for a reply. If nothing has changed, I'll check what is going on.

@hhff
Copy link
Owner

hhff commented Sep 28, 2015

yeah i haven't changed it at all - but I'm guessing it's just an ordering thing - so if the tests explicitly "clicked" the mugs label rather than the first item, we should be cool

@hhff
Copy link
Owner

hhff commented Sep 28, 2015

thank you so so much for working on this @givanse u are the mannnnn

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants