maykinmedia / open-klant

Registratiecomponent voor de opslag en ontsluiting van klantgegevens volgens de Klantinteracties API-specificatie.
Other
2 stars 3 forks source link

:sparkles: [#182] changed internetaken actor field to actoren #218

Closed bart-maykin closed 2 months ago

bart-maykin commented 3 months ago

Fixes #182

Changes

alextreme commented 3 months ago

Besproken met Bart, en de feedback van Anna en de twijfels bij Bart laten zien dat er hier (in hun en mijn ogen onnodige) complexiteit wordt geintroduceerd. Bovendien is in de refinement te weinig aandacht geweest voor het ordering binnen de reeks van actoren die hierdoor ontstaat.

Deze PR gaat onhold voor nu totdat Joeri terug is, de specificatie is te weinig uitgewerkt (en spreekt zichzelf tevens tegen) om hier een klap op te geven

joeribekker commented 2 months ago

Ordering can be on pk of the through model, and not OrderedModel. These are the scenario's. Whether Write is an POST, PUT or PATCH, these rules apply:

1

Note: This case clearly has 2 applications that work differently. The first application set A, B via the new attribute, but the second application uses the deprecated attribute. If this application really sets the value via the old attribute, we have to respect that it now has 1 value.


5