-
Notifications
You must be signed in to change notification settings - Fork 130
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
Not compatible with active_model_serializers 0.9.x #123
Comments
Interesting - I'll take a look into how they handle the detection now and see how hard it would be to integrate into RP |
For people on rails 4.2 looking for easy workaround while issue not fixed:
Be sure to remove this code when issue will be fixed. |
There is a potential integration point for this in |
For a start, given there is a work around for this - that is to say, you can manually set I'll retag this to 1.13, and take a look for that in the near future. Again - patches / specs welcome :) |
Just a note, but I'm experimenting with bringing RP up to spec with AMS 0.10.x which has even further interface changes above and beyond 0.9.x I'll let you know how I get on. |
According to active_model_serializers changelog they removed method
Model#active_model_serializer
. The problem that rocket_pants uses this method if it's present to get serializer from model:The workaround to specify serializer explicitly on
expose
call.The text was updated successfully, but these errors were encountered: