feathers-hooks-common icon indicating copy to clipboard operation
feathers-hooks-common copied to clipboard

Reconsider deprecation of `disable-multi-item-change`

Open 1valdis opened this issue 3 years ago • 1 comments
trafficstars

The deprecation notice suggests usage of multi: true option, but the problem is multi: true has no flexibility at all and sets the behavior for both internal and external calls (see feathersjs/feathers#2546), while the disableMultiItemChange hook be used conditionally, for example iff(isProvider('external'), disableMultiItemChange()). Due to its flexibility, I think its deprecation should be reconsidered and it could be included back to feathers-hooks-common.

1valdis avatar Jan 28 '22 08:01 1valdis