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

Remove Fixture Adapter #3717

Merged

Conversation

greyhwndz
Copy link
Contributor

@bmac: Would be glad to hear some feedback regarding this PR. Thanks!

@bmac
Copy link
Member

bmac commented Aug 30, 2015

@greyhwndz greyhwndz force-pushed the remove-fixture-adapter-references branch from 4d9d3f1 to 188fb14 Compare August 30, 2015 14:51
@greyhwndz
Copy link
Contributor Author

@bmac: Done.

@bmac
Copy link
Member

bmac commented Aug 30, 2015

Oops, looks like the fixture adapter also needs to be emoved from this file: https://github.com/emberjs/data/blob/7b2c70d5e454a485eae4d64c7cdabfa020208b9d/packages/ember-data/lib/adapters.js

@greyhwndz greyhwndz force-pushed the remove-fixture-adapter-references branch from 188fb14 to 60b7ae7 Compare August 30, 2015 15:21
@greyhwndz
Copy link
Contributor Author

@bmac: Just finished removing the references to FixtureAdapter in lib/adapter.js

@greyhwndz
Copy link
Contributor Author

@bmac: This should address issue #3691

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.

3 participants