Skip to content

Upgrade to new jsonapi-deserializable version. #34

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

Merged
merged 1 commit into from
Jul 25, 2017

Conversation

beauby
Copy link
Member

@beauby beauby commented Jul 15, 2017

Pending publishing of jsonapi-rb v0.3.0, containing jsonapi-deserializable v0.2.0.

{ "#{key}_id".to_sym => id, "#{key}_type".to_sym => type }
end
has_many do |_rel, ids, types, key|
key = key.to_s.singularize

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unnecessary spacing detected.
Operator = should be surrounded by a single space.

@@ -3,26 +3,33 @@

module JSONAPI
module Rails
module Deserializable
class Resource < JSONAPI::Deserializable::Resource

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing top-level class documentation comment.

@beauby beauby force-pushed the upgrade-deserializers branch from ab6c4b4 to aac9514 Compare July 15, 2017 05:19
@beauby beauby force-pushed the upgrade-deserializers branch 3 times, most recently from 93e38ed to 2e0a325 Compare July 25, 2017 06:11
@beauby beauby force-pushed the upgrade-deserializers branch from 2e0a325 to 1e83942 Compare July 25, 2017 06:18
@beauby beauby changed the title Upgrade to upcoming jsonapi-deserializable version. Upgrade to new jsonapi-deserializable version. Jul 25, 2017
@beauby beauby merged commit 1b3af0a into master Jul 25, 2017
@beauby beauby deleted the upgrade-deserializers branch July 25, 2017 06:50
mdeutsch pushed a commit to patientslikeme/jsonapi-rails that referenced this pull request Jul 25, 2017
* upstream/master:
  Conditionally include controller mixin (jsonapi-rb#40)
  Add initializer generator. (jsonapi-rb#39)
  Add ActiveSupport::Notifications instrumentation. (jsonapi-rb#41)
  Add library configuration object (jsonapi-rb#17)
  Rename JSONAPI::Rails::ActionController to JSONAPI::Rails::Controller. (jsonapi-rb#38)
  Add hook for default jsonapi object. (jsonapi-rb#37)
  Add hook for default exposures. (jsonapi-rb#36)
  Add hook for automatic pagination links. (jsonapi-rb#35)
  Upgrade to new jsonapi-deserializable version. (jsonapi-rb#34)
  Prepare for v0.2.1 release
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