nekochans / lgtm-cat-frontend

lgtm-cat(サービス名 LGTMeow https://lgtmeow.com) のフロントエンド用
https://lgtmeow.com
21 stars 2 forks source link

Middlewareでエラーを表示させる際にMiddlewareから直接JSONを返すように変更 #285

Closed keitakn closed 6 months ago

keitakn commented 6 months ago

issueURL

https://github.com/nekochans/lgtm-cat-frontend/issues/228

関連 URL

https://vercel.live/open-feedback/lgtm-cat-frontend-git-feature-issue228-nekochans.vercel.app?via=pr-comment-visit-preview-link&passThrough=1

このPRで対応すること / このPRで対応しないこと

https://github.com/nekochans/lgtm-cat-frontend/issues/228 の完了の定義を満たす為の実装は全てこのPRで実装する。

Storybook の URL もしくはスクリーンショット

UI変更は実施していないので省略

変更点概要

タイトルの通り。優先度は低かったがこれを放置しておくとAppRoterへの移行時にRouteHandlerを作成しないといけないのでこのタイミングで対応する事にした。

レビュアーに重点的にチェックして欲しい点

特になし

補足情報

特になし

vercel[bot] commented 6 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
lgtm-cat-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 15, 2024 2:49pm
codecov[bot] commented 6 months ago

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (b9d862c) 28.52% compared to head (4518e19) 28.57%.

Files Patch % Lines
src/middleware.ts 0.00% 4 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #285 +/- ## ========================================== + Coverage 28.52% 28.57% +0.04% ========================================== Files 215 214 -1 Lines 6707 6699 -8 Branches 270 269 -1 ========================================== + Hits 1913 1914 +1 + Misses 4634 4626 -8 + Partials 160 159 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.