search

rtablada / eloquent-ember

Makes Ember APIs super easy to build in Laravel
33 stars 3 forks source link

Updating to latest laravel and ember data #6

Open Globegitter opened 10 years ago

Globegitter commented 10 years ago

Due to the composer.json this is not working with the latest laravel 4.1.x. Would be great if that could be updated. Also, ember-data now expects pluralized words for relationships rather than 'relationships_id', if that makes sense. See here: http://emberjs.com/guides/models/connecting-to-an-http-server/#toc_json-conventions I am just trying to update it myself, bu don't seem to getting it to work. Would really appreciate any help.

rtablada commented 10 years ago

I will have to look, and tag some releases. Will definitely update to use Laravel ~4.

Globegitter commented 10 years ago

Wow, thanks for the quick reply. That is some good news.

Globegitter commented 10 years ago

@rtablada Sorry made a mistake with the naming conventions. I just realized with the latest ember-data beta and ActiveModelAdapter you still need 'relationship_ids'. I just returned both because I didn't know at the beginning and somehow mixed that up.

Edit: I am not sure if the naming conventions are as shown in the link for the RESTAdapter though, or if that is a mistake on the website.

andragan commented 10 years ago

@Globegitter , @rtablada I think it should be {relationship}s instead of {relationship}_ids. also tried to update it myself, but it doesn't work. the related attribute does not get included in the response once I change it to {$relation}s ,I think something catches attributes with {$relation}s index and kicked it out of the model