search

mhgbrown / cached_resource

Caching for ActiveResource
MIT License
80 stars 28 forks source link

Chained calls do not work like it does in `active_resouce` #57

Closed jlurena closed 7 months ago

jlurena commented 8 months ago

Example:

MyActiveResource.where(name: 'bob').where(last_name: 'the builder')

throws an error because the previous params { name: 'bob' } is not assigned to the original_params https://github.com/rails/activeresource/blob/df28dee231f8573c18d9e091c7fd7bc6d6679263/lib/active_resource/base.rb#L1097

I can also create PR if its agreed this should be fixed.

mhgbrown commented 8 months ago

I agree, go for it.

jlurena commented 7 months ago

Created PR: https://github.com/mhgbrown/cached_resource/pull/59 @mhgbrown

jlurena commented 7 months ago

Additional: https://github.com/mhgbrown/cached_resource/pull/60