search

gtalarico / pyairtable

Python Api Client for Airtable
https://pyairtable.readthedocs.io
MIT License
765 stars 138 forks source link

Ignore extra fields in ORM (fixes #190, option 1) #250

Closed mesozoic closed 1 year ago

mesozoic commented 1 year ago

I'm not sure there's a valid use case for the ORM to fail when it gets a field it doesn't recognize. If someone has a use case in mind, we can always make this a configuration flag on Meta later, but I think it should be permissive by default.

codecov-commenter commented 1 year ago

Codecov Report

Merging #250 (a0c96de) into main (b06baa6) will increase coverage by 0.46%. The diff coverage is 92.85%.

:exclamation: Current head a0c96de differs from pull request most recent head dda7935. Consider uploading reports for the commit dda7935 to get more accurate results

@@            Coverage Diff             @@
##             main     #250      +/-   ##
==========================================
+ Coverage   87.61%   88.07%   +0.46%     
==========================================
  Files          14       15       +1     
  Lines         662      671       +9     
==========================================
+ Hits          580      591      +11     
+ Misses         82       80       -2
Impacted Files Coverage Δ
pyairtable/orm/model.py 86.79% <50.00%> (+1.47%) :arrow_up:
pyairtable/testing.py 100.00% <100.00%> (ø)