-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
reply handling #3099
Comments
Sigh, I suspected this will happen. I'll issue a revert asap. |
Confirmed, the change is semver-major. |
@climba03003 I actually went for a better solution, I fixed the underlining issue and make Next.js tests passing again. |
check out #3099 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
馃挜 Regression Report
I think it should be related to the reply simplification.
fastify/fastify-nextjs#260
Not sure if it is only test case related only or production related. I am not using this plugin but it is worth knowing the test is currently failed.
Last working version
Worked up to version: 3.15.1
Stopped working in version: 3.16.1
The text was updated successfully, but these errors were encountered: