-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
docs(adapters/fastify): usage of routerOptions #6906
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
docs(adapters/fastify): usage of routerOptions #6906
Conversation
The latest updates on your projects. Learn more about Vercel for GitHub.
|
@theoludwig is attempting to deploy a commit to the trpc Team on Vercel. A member of the Team first needs to authorize it. |
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
@trpc/client
@trpc/next
@trpc/react-query
@trpc/server
@trpc/tanstack-react-query
@trpc/upgrade
commit: |
Set as draft because TypeScript types have not yet been updated. |
Since Fastify v5.5.0, there is the following deprecation warning:
Related fastify/fastify#6282, fastify/fastify#5985
🎯 Changes
Update the Fastify adapter documentation to fix the deprecation warning.
Should we put the 2 possible ways for both v5.4 and prior Fastify versions, and for v5.5.0 and above, or it's fine to only put the "latest" way (like how it is done currently in this PR)?
✅ Checklist