Skip to content

Conversation

@colstrom
Copy link
Contributor

This fixes a NameError that prevents related from being loaded.

`activemodel-serializers-xml` is required for this gem to load, but is not currently included in `activemodel`. It was extracted for ActiveModel v5.x.

Either adding it to the dependencies, or applying a `~> 4.2` constraint to the `activemodel` dependency resolves this. I've opted for the former in this patch.
Adding it to the `gemspec` is half the battle, this is the other half. It needs to be required separately, as it has been extracted from ActiveModel.
@colstrom colstrom changed the title Resolve uninitialized constant ActiveModel::Serializers::Xml Resolve uninitialized constant ActiveModel::Serializers::Xml Aug 22, 2016
@niho
Copy link
Owner

niho commented Sep 1, 2016

@colstrom If you fix the conflict I will merge this pull request ASAP.

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