Closed keitakn closed 6 months ago
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 5:46am |
codecov[bot]
commented
6 months ago Attention: 3631 lines in your changes are missing coverage. Please review.
Comparison is base (
a20346a) 59.66% compared to head (a2897c9) 28.52%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
issueURL
https://github.com/nekochans/lgtm-cat-frontend/issues/283
関連 URL
https://lgtm-cat-frontend-git-feature-issue283move-com-973b53-nekochans.vercel.app/
このPRで対応すること / このPRで対応しないこと
https://github.com/nekochans/lgtm-cat-frontend/issues/283 のDoneの定義の通り、
@nekochans/lgtm-cat-uiのComponentが移行されている事。Storybook の URL もしくはスクリーンショット
https://622b6c5dc31e9e003a111eb5-ptlegmaxpu.chromatic.com/
変更点概要
lgtm-cat-uiのComponentを移植。基本的にそのまま移行しているが一部設計を変更したほうが良い箇所は変更を実施している。本件は最初 https://github.com/nekochans/lgtm-cat/issues/35 のエピックで進める予定だったが最新のNext.jsにバージョンアップ、設定ファイルの変更を行ったところBuildエラーが出るようになってしまったので先にこちらを実施する事にした。
lgtm-cat-uiのComponentを全て移行したのでtsconfig.jsonのmoduleResolutionをbundlerに変更。これはNext.jsの最新のテンプレートの設定に合わせた形になっている。レビュアーに重点的にチェックして欲しい点
特になし
補足情報
BuildされたStorybook上だとMSWが上手く動作していない事が分かった。見た感じ設定は合っているっぽいのでStorybookのmsw-storybook-addonの不具合かもしれないのでしばらく様子を見る。storybook dev -p 6008でローカルでの動作確認だと正常に動作する。↑CI上で実行するNode.jsのバージョンを20系に変更したらMockが正常に動作するようになった。