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

[WIP] Removes deprecation from upcoming Ember 2.12 release. #240

Merged
merged 1 commit into from
Jan 30, 2017

Conversation

workmanw
Copy link
Contributor

Let's hold on merging this until after Ember-Data's change lands in their beta channel. See: emberjs/data#4743 .

@workmanw workmanw mentioned this pull request Jan 25, 2017
3 tasks
@workmanw workmanw force-pushed the factoryFor branch 4 times, most recently from a39aabe to 54344d8 Compare January 26, 2017 22:14
@workmanw
Copy link
Contributor Author

@jakesjews The upstream commit was cherry-picked into the beta channel. :). Okay to merge this one? After that, I'll rerun the tests with the other PR and then rebase it and we should be passing for the beta :).

@workmanw workmanw merged commit 36c143b into master Jan 30, 2017
@jakesjews
Copy link
Contributor

@workmanw sorry about the late response. Looks good 👍

@workmanw
Copy link
Contributor Author

@jakesjews No worries :) If you have a moment can you take a look at #232.

@jakesjews
Copy link
Contributor

@workmanw will do

@workmanw workmanw deleted the factoryFor branch January 31, 2017 02:56
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