remove all onSuccessCallback params

[jamieknight-db]

Removing all onSuccessCallback parameters in hooks, they are not needed and can be handled in mutation onSuccess instead.

[jamieknight-db]

@xx-db please review when you have a moment

[yc-shawn]

Thanks!

Assuming you already manually tested those

[Romanize]

Looks nice, can you double check the invalidations are working properly?

[jamieknight-db]

Thanks!

Assuming you already manually tested those

@yc-shawn yes they have been tested and working as expected

[jamieknight-db]

Looks nice, can you double check the invalidations are working properly?

@Romanize everything is working as it was before, but I noticed the tree view browser is not updating after any of the invalidateQueries (which was an existing issue). It's bothering me so I am looking into a fix.

[jamieknight-db]

Going to merge this and circle back to the issue of refreshing the tree view. I haven't been able to find a solution.