Replies: 2 comments 1 reply
-
cc: @GP4cK @smolinari |
Beta Was this translation helpful? Give feedback.
1 reply
-
LGTM. 😁 Scott |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
It's currently already placed inside the code as a TODO but wanted to check what others think about this.
Things that I notice in our own API's is that in almost all relations we set
disableUpdate
anddisableRemove
, recently theupdate.<options>
andremove.<options>
(#123) where added to also support relations decorators, descriptions etc.My main idea is to align the update/remove with the aggregate and make it opt-in, this would change then from this:
To this:
Beta Was this translation helpful? Give feedback.
All reactions