You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In mixins.py there is AirtableMixin.create_record() and AirtableMixin.update_record().
These should ideally be merged together. Originally there were valid reasons for keeping these separate, but after further development they can merged.
Merging the two methods into a create_or_update() method would reduce logic in AirtableMixin.save() and we could likely remove some of the unit tests and merge those together as well.
The text was updated successfully, but these errors were encountered:
In
mixins.py
there isAirtableMixin.create_record()
andAirtableMixin.update_record()
.These should ideally be merged together. Originally there were valid reasons for keeping these separate, but after further development they can merged.
Merging the two methods into a
create_or_update()
method would reduce logic inAirtableMixin.save()
and we could likely remove some of the unit tests and merge those together as well.The text was updated successfully, but these errors were encountered: