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
Is your feature request related to a problem? Please describe.
One can pass query options to the list, but not mutation options to the BulkUpdateButton
Describe the solution you'd like
It would be nice if the buttons accepted not only mutationMode, but also mutationOptions prop the same way that BulkDeleteButton and Edit components do.
Describe alternatives you've considered
Maybe it doesn't have to be buttons, but list prop could accept the parameter and button scould somehow get it from the context instead.
The text was updated successfully, but these errors were encountered:
I think we should definitely add mutationOptions to BulkUpdateButton, especially provided that other similar buttons (BulkDeleteButton, DeleteButton) already support it.
I think it's best to set the meta prop at the button level, as it allows for more flexibility in case you need to pass a specific value for this case, besides I don't see what context could hold this value.
I added support for the mutationOptions in this pull #9035 I tried to follow implementation of BulkDeleteButton so that they are the same. Please let me know if the code is not up to standard I will update it.
Is your feature request related to a problem? Please describe.
One can pass query options to the list, but not mutation options to the BulkUpdateButton
Describe the solution you'd like
It would be nice if the buttons accepted not only mutationMode, but also mutationOptions prop the same way that BulkDeleteButton and Edit components do.
Describe alternatives you've considered
Maybe it doesn't have to be buttons, but list prop could accept the parameter and button scould somehow get it from the context instead.
The text was updated successfully, but these errors were encountered: