The tests are failing. Likely need some updates since this was last looked at.

@javierjulio This is now a breaking change.scoped_collection is not raisingPunditError anymore.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.11%. Comparing base(7117d59) to head(282f4f9).

@@           Coverage Diff           @@##           master    #7511   +/-   ##=======================================  Coverage   99.11%   99.11%           =======================================  Files         141      141             Lines        4069     4087   +18     =======================================+ Hits         4033     4051   +18  Misses         36       36

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report?Share it here.

@javierjulio took me some time to build a proper spec. Now is ready for review.
For the record, tests were failing because the code assumes that we can always get a scoped collection. But the template app define a default scope nor a policy for Tagging class (which is fine). That makesscoped_collection to raise an error.
I didn't want to investigate why raising an error is the expected result. In the context of building filters, I find it good enough to fallback to "all" scope if this happens.

What is the benefit of adding this? If I understand right it only affects default filters, so if you set filters explicitly which is best, this would not apply. Or I guess it would since there is an update toadd_filter method. I'm not convinced on this change.

@javierjulio The change affects filters and default filters. I added more test scenarios to demonstrate the usage. I understand the previous commit was misleading.
If you use an implicit filter collection, we make sure it is accessible by the user. That's the main idea.