Skip to content

Pr/153#155

Merged
dgeb merged 5 commits intomasterfrom
pr/153
Apr 8, 2015
Merged

Pr/153#155
dgeb merged 5 commits intomasterfrom
pr/153

Conversation

@lgebhardt
Copy link
Copy Markdown
Contributor

Adds some tests to #153

adamworrall and others added 5 commits April 8, 2015 00:39
POST with only 'type' key is not triggering a save on the activerecord model, but no errors are reported. This leads to an incorrect 201 status.
Resource#create_model instantiates an activerecord model, but doesn't trigger a save. The save comes in _replace_fields, but if field_data[:attributes] is empty it never sets @save_needed.

I'm sure this isn't the best way to solve this issue, but it works for now.
dgeb added a commit that referenced this pull request Apr 8, 2015
@dgeb dgeb merged commit 5a6e5b4 into master Apr 8, 2015
@lgebhardt lgebhardt deleted the pr/153 branch April 8, 2015 15:01
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