search

mwarger / tmdb-watchlist-prisma

NX monorepo showing the TMDB Watchlist mobile app with Expo, tRPC, Next, and Prisma
70 stars 8 forks source link

Add `trpc-openapi`? #2

Open jlalmes opened 2 years ago

jlalmes commented 2 years ago

🚀 Great project @mwarger! Looks like this is a great starting place for tRPC developers to understand how they could structure their apps using an NX-powered monorepo.

Taking a little scan through the api/trpc-server library it looks like you've already added output schemas to lots of your procedures. Any interest in adding trpc-openapi too? It should be pretty simple, I'm happy to assist!

Full disclosure: I am the maintainer of trpc-openapi. I would love to point people toward a real world monorepo example.

mwarger commented 2 years ago

Thanks for the kind words @jlalmes - I appreciate all the help you provide to tRPC!

Funny enough, I actually originally had that in the massive conference talk example I used and I provided it as an example of using swagger. I took it out of this one to simplify. I'm not opposed to adding it. Could you possibly assist with a PR?

And come to think of it, I did try to integrate trpc-openapi with react-admin at one point, but didn't manage to get it working. This might be a good opportunity to give that another shot.