Did I overly abstracted ?

[naveen1337]

I have a user management table to build, it has lot of feature like filtering updating status, I like to keeping all function and API calls in page component, and that's makes by code lines long, so I started abstract all the code into custom hooks.

export function useMemberDelete(toast: any, toastRef: any, membersRf: any) {
    const deleteMemberReq = useMutation(deleteMemberByIdRemote, {
        onSuccess: (data, vars) => {
            console.log(data);
            if (data.txn) {
                membersRf()
                toast.success('User Deleted successfully', { id: toastRef.current })
            }
            else {
                toast.error(data.res, { id: toastRef.current })
            }
        },
        onError: (err) => {
            toast.error('something went wrong !', { id: toastRef.current })
        },
        onMutate: () => {
            const tid = toast.loading('loading ...')
            toastRef.current = tid
        }
    })
    return {
        deleteReq: deleteMemberReq.mutate,
        deleteReq_l: deleteMemberReq?.isLoading,
        deleteReq_e: deleteMemberReq.isError,
        deleteReqRes: deleteMemberReq?.data,
    }
}

I still keep thinking did I overly abstracted a unnecessary things. it scares me is it a recommended way or is there any hidden problem will arise from future

Give me your opinion

Thanks