Add attribute use_field_ids to orm.Model.Meta

[BAPCon]

Resolves #351
Resolves #187

Adds wrapper @modify_kwargs that handles kwarg sanitization for:

• Model.all()
• Model.first()

cell_format and return_fields_by_field_id are removed.

If Model.Meta.use_field_ids=True, adds return_fields_by_field_id=True to above function calls.

I figure adding a dedicated wrapper is easier to read and is easier to manage if more sanitization is needed in the future.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (eef7b6f) to head (64f9341).
Report is 7 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #355   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           25        25           
  Lines         2313      2320    +7     
=========================================
+ Hits          2313      2320    +7     

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

[mesozoic]

Thanks for contributing a fix! I like this idea and I think it'll make the ORM more useful. I have a few suggestions below but otherwise I'm looking forward to adding this.

[mesozoic]

This is looking good! The test seems to satisfy all the conditions we need to test, and it looks like we've maintained 100% coverage. A couple more pieces of feedback here, but nothing major.

[mesozoic]

Looks great! Thank you for contributing a new feature!