keitakn commented 6 months ago

issueURL

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

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

現在の本番環境と同じく i18n対応が実施する。

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

https://622b6c5dc31e9e003a111eb5-wistwfxmsw.chromatic.com/

変更点概要

Pages Routerの時は next.config.js での設定と next/link と locale を使って簡単に実現出来ていたが App Router だと middleware 等で自前で実装しなけばならないので以下の条件を満たす実装を行った。

言語の切り替えはパスで行う（言語の切り替えは言語メニューにて行う）
有効な言語は日本語（ja）と英語（en）
defaultの言語は ja
/ja にリクエストされた場合は / にリダイレクトする
現在選択中の言語に合わせて各リンク先のURLを変更する

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

特になし

補足情報

インラインコメントに記載。

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 23, 2024 4:00pm

codecov[bot] commented 6 months ago

Codecov Report

Attention: Patch coverage is 17.62115% with 374 lines in your changes are missing coverage. Please review.

Project coverage is 24.58%. Comparing base (378ea1c) to head (3067190).

Files	Patch %	Lines
src/components/Header/LanguageMenu.tsx	0.00%	42 Missing and 1 partial :warning:
src/middleware.ts	0.00%	43 Missing :warning:
src/app/en/error.tsx	0.00%	35 Missing and 1 partial :warning:
src/app/en/layout.tsx	0.00%	35 Missing and 1 partial :warning:
src/app/en/privacy/page.tsx	0.00%	31 Missing and 1 partial :warning:
src/app/en/terms/page.tsx	0.00%	31 Missing and 1 partial :warning:
src/app/en/maintenance/page.tsx	0.00%	17 Missing and 1 partial :warning:
src/app/en/not-found.tsx	0.00%	17 Missing and 1 partial :warning:
src/app/en/page.tsx	0.00%	12 Missing and 1 partial :warning:
src/features/url.ts	75.92%	13 Missing :warning:
... and 16 more

Additional details and impacted files

```diff @@ Coverage Diff @@ ## release/v2.8.0 #291 +/- ## ================================================== - Coverage 24.82% 24.58% -0.24% ================================================== Files 215 223 +8 Lines 7735 8122 +387 Branches 271 298 +27 ================================================== + Hits 1920 1997 +77 - Misses 5655 5957 +302 - Partials 160 168 +8 ```

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

nekochans / lgtm-cat-frontend

App Routerのi18n対応を実施 #291