claytondaley / drf-writable-nested

Writable nested model serializer for Django REST Framework
Other
1 stars 2 forks source link

Refine Semantics (plus fixes) #4

Closed claytondaley closed 4 years ago

claytondaley commented 4 years ago

This is a quick warning to anyone using my fork (and hopefully watching the repo) that I'm making backwards-incompatible adjustments to the semantics of my new-style Mixins. Specifically, Get now means "get but don't update" and Update means "get and update". As a result, the old GetOrCreate behavior is now found in UpdateOrCreate. Please update your models accordingly.

This PR also includes some fixes I discovered for some edge cases (e.g. source='*'). There may be unintended side-effects so please test the package ASAP if you are not bound to a specific commit. If you depend on this repo, I suggest a strategy I use internally:

This ensures that your build process downloads a predictable release and changes to master can be tested before they are incorporated. (and please let me know if you are using the fork!)