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
I have opened a pull-request. During my investigation in the code, it seems that there a minimum of 3 different usages of an onSuccess property in different locations. Additionally, there were a lot of locations where basically no typing was present (everything was any). I tried to remove some abiguity by defining more types.
I was not quite sure how to test that, so I am happy for suggestions on how to improve the PR.
According to the CreateProps typings, the onSuccess callback has the type
(data: Record) => void
.However, it seems that the true implementation in the useCreateController the type
({ data } : { data: Record}) => void
is used.Right now the two are incompatible. I would suggest to change the type as this breaks no implementation based on the current behaviour?
The text was updated successfully, but these errors were encountered: