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

Passing classes to store methods has been removed #272

Closed
20v100 opened this issue Jun 27, 2015 · 7 comments
Closed

Passing classes to store methods has been removed #272

20v100 opened this issue Jun 27, 2015 · 7 comments

Comments

@20v100
Copy link

20v100 commented Jun 27, 2015

Hello,

Just updated to Ember-Data 1.13 and Ember 2.0.0-beta-1 and for the following error:

Uncaught Error: Assertion Failed: Passing classes to store methods has been removed. Please pass a dasherized string instead of krc@model:user:

After investigation, I think the error is between Ember-Data and Ember-Fire.

My code is:

`return this.store.findAll('user').then(function(user) {...``

@tstirrat
Copy link
Contributor

Please use the master branch for ember data 1.13.x compat. (see #138 for how to use a branch)

Duplicates #261

See also #267

@tstirrat
Copy link
Contributor

Reopening for visibility

@tstirrat tstirrat reopened this Jul 24, 2015
@IsaiahJTurner
Copy link
Contributor

Could you explain what your message above means? The latest release version doesn't work?

@tstirrat
Copy link
Contributor

I am reopening this issue so that it is clear it is a known issue and we don't get more duplicates.

We are working on the EmberFire 1.13.x compatible release. The code is done, the documentation is being reviewed now.

Use master branch until we do that:

ember new app
cd app
ember install firebase/emberfire#master

@tstirrat tstirrat added this to the 1.13.0 Compat milestone Jul 27, 2015
@IsaiahJTurner
Copy link
Contributor

Ahh, thank you! For some reason I was thinking the latest release was master.

Does this mean anything to you? It looks like something is expecting an array and not getting one. It might be a personal code issue, not a EmberFire issue; I'll keep looking into it.

Edit: think I found the issue. What serializer should I be using?
Edit: Confirmed, switching to the FirebaseSerializer works.
screen shot 2015-07-27 at 1 51 02 pm

@tstirrat
Copy link
Contributor

Looks like it's going through the JSON-API serializer.. we dont use that for emberfire (we are based off the JSONSerializer).

Check if you have a serializer in app/serializers/application.js or app/serializers/<model>.js

If you do have one, usually you just want to delete the file and let emberfire use the default. If you have some need for a custom serializer, make sure it extends from the right one:

// app/serializers/post.js
import FirebaseSerializer from 'emberfire/serializers/firebase';

export default FirebaseSerializer.extend({
});

@tstirrat
Copy link
Contributor

1.5.0 is released :)

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

No branches or pull requests

3 participants