Skip to content

types: request route schema might be undefined#5394

Merged
mcollina merged 3 commits intofastify:mainfrom
nflaig:schema-type
May 7, 2024
Merged

types: request route schema might be undefined#5394
mcollina merged 3 commits intofastify:mainfrom
nflaig:schema-type

Conversation

@nflaig
Copy link
Copy Markdown
Contributor

@nflaig nflaig commented Apr 9, 2024

Closes #5393 by marking route schema type as optional

Checklist

Co-authored-by: Manuel Spigolon <behemoth89@gmail.com>
Signed-off-by: Nico Flaig <nflaig@protonmail.com>
Signed-off-by: Nico Flaig <nflaig@protonmail.com>
Copy link
Copy Markdown
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I prefer #5395 to this.

@nflaig
Copy link
Copy Markdown
Contributor Author

nflaig commented Apr 23, 2024

Closing in favor of #5395

@nflaig nflaig closed this Apr 23, 2024
@nflaig nflaig reopened this May 6, 2024
Copy link
Copy Markdown
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

cc @Eomm

@mcollina mcollina merged commit 445e41a into fastify:main May 7, 2024
@github-actions
Copy link
Copy Markdown

github-actions bot commented May 8, 2025

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Route schema might be undefined

3 participants