mhgbrown / cached_resource

Caching for ActiveResource
MIT License
80 stars 28 forks source link

[ISSUE-62]: Avoid cache nil or empty array responses #63

Closed italijancic-th closed 5 months ago

italijancic-th commented 5 months ago

ISSUE 62

Resolves #62

Changes

Refactor find_via_reload(key, *arguments)

Test results

Screenshot 2024-03-21 at 11 14 31
jlurena commented 5 months ago

@mhgbrown any chance we can get this merged in?

mhgbrown commented 5 months ago

@jlurena totally, will merge. Can you not as a collaborator? I'll work on the deploy.

mhgbrown commented 5 months ago

It's out! Thank you @italijancic-th

italijancic-th commented 5 months ago

It's out! Thank you @italijancic-th

Thank you @mhgbrown letting me contribute, and thanks @jlurena for the code reviews!