-
-
Notifications
You must be signed in to change notification settings - Fork 37
Modernizing to tRPC v10 #24
base: main
Are you sure you want to change the base?
Conversation
I have a repo you could look at for reference: https://github.com/juliusmarminge/turbo-expo-next-starter |
Please let me know if I've missed anything @KATT 🙏🏽 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On my phone, excited to try this out ASAP 😋
@@ -31,5 +128,3 @@ export type inferMutationOutput< | |||
export type inferMutationInput< |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add the remaining three inference helpers?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Which helpers do you mean? @juliusmarminge
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are changing with the new major
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We'll do something like this instead:
import { inferProcedureOutput } from '@trpc/server';
type PostByIdOutput = inferProcedureOutput<AppRouter['post']['byId']>;
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ooh thats nice. No need for the helpers then. When will this be a thing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a thing in V10 already
@@ -1 +1 @@ | |||
export * from '@zart/react/trpc'; | |||
export * from '@zart/react-native/trpc'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was causing an error when running expo
I'm trying to incorporate solito as well, should I try to upgrade expo as well? @KATT |
I would be careful about expo upgrades. Make sure the app works before and after :D |
Maybe I can try on a follow up Pr |
Expo just launched v46 beta with React 18 so I would hold off until thats stable |
Is it working in expo now? Could you update the readme if something has changed in how you use it? |
It should be working yeah. Nothing has changed AFAIK |
Expo has released 46 now 🙂 |
Hey there, any news on this? |
A bit of self-promotion for a more minimal setup: create-t3-turbo |
__ Im afraid my expo knowledge is limited, anyone feel free to take over my work on this PR. It's on a mergeable state if you ask me. Maybe upgrading expo can be a follow up. |
I was wondering why it is necessary to have the |
TBH I just kept what was already there |
refs trpc/trpc#2118
next.config.js