Skip to content
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

Upgrade packages #1

Merged
merged 3 commits into from Nov 9, 2022
Merged

Conversation

junkor-1011
Copy link
Contributor

@joe-king-sh さん

https://dev.classmethod.jp/articles/fastify-zod-openapi/ ですが、非常に興味深い記事で参考になりました。ありがとうございます。
ただ、記事の執筆日から現在にかけて@fasify/fastify-swaggerの仕様がv7->v8で変更になったことが原因で、
記事の手順通りではswagger-uiを表示出来ない現象が起こり、少しハマりました。
(※新しく出来た@fastify/fastify-swagger-uiを追加してsrc/app.tsの処理を微修正する必要がありました。)

上記に事象に対応しつつ、現在の最新のパッケージの動かすための修正についてプルリクの形でご参考としてお伝えします。
(特にマージ頂きたい訳ではなく、記事だとコメント欄が無い?のとtwitterなどだと文字数制限で説明が難しそうだったため、というだけの理由です。。。)

結論としては、

  • 最新のパッケージを使う場合はmigration guideに従って処理を少し改める
    または、
  • @fasify/swaggerのバージョンを7系を明示的に指定するようにする
    のいずれかの対応が必要な旨を記事に追記頂けるとより良いのではないかと感じました。

ご参考頂けますと非常に幸いです。よろしくお願いいたします。

@joe-king-sh
Copy link
Owner

@junkor-1011 さん

ご連絡いただきありがとうございます。
公開していた記事がそのままで検証できず、お手数おかけいたしました🙇‍♂️

@fastify/fastify-swaggerのv8から、@fastify/fastify-swagger-uiが必要になったのですね。
PullRequestまで作成していただき大変助かります。

元記事の方も併せて修正させていただきます。

ありがとうございました!

@joe-king-sh joe-king-sh merged commit df0ea27 into joe-king-sh:main Nov 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants