electron-trpc icon indicating copy to clipboard operation
electron-trpc copied to clipboard

TRPC 11.x.x (next) introduces breaking changes that will break electron-trpc

Open mat-sz opened this issue 11 months ago • 5 comments

One of the breaking changes involves transformers, the error manifests itself on client side as:

Uncaught TypeError: Cannot read properties of undefined (reading 'serialize')

With this line being at fault: https://github.com/jsonnull/electron-trpc/blob/784bbb0cf45b0304c9f55bd9827a51c8dd3a5b6a/packages/electron-trpc/src/renderer/ipcLink.ts#L97

I've forked the repository and resolved the compatibility issues with the next version of TRPC: https://github.com/mat-sz/trpc-electron/commit/994bd4132850aedf5031ab402131bcfffa80825e

Since this is an unstable version of TRPC, I'm not submitting this as a pull request. If you'd like me to submit that in the future when 11.x.x becomes stable, please let me know; otherwise I'll continue maintaining this as a fork.

mat-sz avatar Feb 27 '24 18:02 mat-sz

Hey, thanks so much for doing this! ❤️

I'm happy to accept your changes if you'd like to open a PR. Otherwise, I'll catch up with the latest TRPC in this repo soon.

jsonnull avatar May 03 '24 21:05 jsonnull

@jsonnull Hey, thanks for getting back to me, I'll create a PR tomorrow with all of those changes.

mat-sz avatar May 03 '24 21:05 mat-sz

@jsonnull Created a PR - #194

mat-sz avatar May 04 '24 11:05 mat-sz

Just bumping this issue if that's ok. I know there's a PR already and adding support for TRPC v11+ and thus React Query v5 would be really welcome indeed :) I'm happy to help and test!

theorib avatar Aug 27 '24 18:08 theorib

This PR worked for me with no issues. It'd be great if it could be merged. For now, I just copied the files locally and applied the patch. Otherwise, I would use this fantastic package!

dsaltares avatar Sep 13 '24 10:09 dsaltares