warning to install ransack polute logs

[gagalago]

Expected Behavior

no log about ransack if we don't use filtering and don't have ransack installed

Actual Behavior

we have a warning when this file is loaded by jsonapi.rb gem

Steps to Reproduce the Problem

1. start the app without ransack gem installed
2. look at logs

Proposition of solution

One way to fix this is to not load this file on the gem and let the user loading it when they need it (as already explained in the readme). That will need to bump the major version of the gem but should not affect people that already followed the readme.

What do you think? do you see another way to fix it? it's not a bug but a feature and you prefer to keep it as is?

[stas]

Ah, thanks for the report!

This is related to #64 change, and I honestly would like to keep it like that, at least until a larger release is out.
The idea is that folks upgrading to this ransack-less version, should be warned about the change in deps.

Is it ok if we keep it like this until the next major release?

[gagalago]

we can live with it. it create confusion every 2-3 months in my team when someone new look at logs so not an hard blocker for me.

Another option that I see is to move the require in the extract_attributes_and_predicates to lazy require it and add a log as a post-install message to keep warning people.

what do you think about this second proposition?

[stas]

@gagalago that works too. Happy to merge it if you have some time to submit the change 🙏

[stas]

Fixed in 1f60bdf