# Liem確認依頼: クライアント修正指示とPR確認

作成日: 2026-05-20  
対象リポジトリ: `ryo2gxo/moro`  
対象ブランチ: `fix/client-feedback-next-20260516`

## 目的

クライアントからの受入テスト修正指示を、GitHub上のPRと照合して確認してください。

特に、以下を確認してください。

1. クライアント修正指示の内容が漏れなく整理されているか。
2. 既存PRに含まれている修正範囲が正しいか。
3. 未コミットの実装修正をPRへ含めるべきか。
4. 仕様確認が必要な項目について、クライアントへ回答・確認すべき内容に問題がないか。
5. 本番反映してよい順序に問題がないか。

## 参照する修正指示

クライアント指示書:

- `public/requests/client-feedback-20260428.md`

回答・仕様確認メモ:

- `public/requests/client-feedback-answers-20260428.md`

現在の指示書ステータス:

- 全81件
- 対応済み: 77件
- 仕様確認: 4件
- 明示的な未対応: 0件

ただし、対応済みメモがあるものの、実装修正が未コミット/未マージのものがあります。下記を必ず確認してください。

## 確認対象PR

### PR #11

タイトル: `fix: apply client feedback cleanup`  
ブランチ: `fix/client-feedback-kirei-cleanup`  
状態: OPEN

確認してほしい内容:

- 着物相談室から着物相談窓口への名称統一
- メール文面の軽微修正
- `routes/api.php` の不要/重複ルート削除
- developへ先に入れて問題ないか

### PR #12

タイトル: `PAY.JP EMV 3Dセキュア対応`  
ブランチ: `fix/payjp-3ds-20260516`  
状態: OPEN

確認してほしい内容:

- PAY.JPのEMV 3Dセキュア対応として妥当か
- カード登録時の3Dセキュア認証フローで問題ないか
- 既存カード、注文時カード選択、保管サービスカード登録への影響
- migration適用後の既存データ互換性
- 本番PAY.JP設定で疎通確認が必要な箇所

### PR #13

タイトル: `クライアントフィードバック確認事項の回答整理`  
ブランチ: `fix/client-feedback-next-20260516`  
状態: OPEN

現状のPR #13に含まれるファイル:

- `public/requests/client-feedback-20260428.md`
- `public/requests/client-feedback-answers-20260428.md`

注意:

PR #13は現時点ではドキュメント整理のみです。実装コードの大半は未コミットのため、GitHub上のPR差分にはまだ出ていません。

## 未コミットだがPRに含める候補の実装修正

`fix/client-feedback-next-20260516` の作業ツリーには、以下の実装修正候補があります。

主な変更対象:

- `app/Http/Controllers/Admin/ProductController.php`
- `app/Http/Controllers/Admin/StorageContractController.php`
- `app/Http/Controllers/Admin/TemplateController.php`
- `app/Http/Controllers/CartController.php`
- `app/Http/Controllers/ContactController.php`
- `app/Http/Controllers/UserController.php`
- `app/Services/StorageChargeService.php`
- `resources/js/backend/views/order/Edit.vue`
- `resources/js/backend/views/order/List.vue`
- `resources/js/backend/views/owner-reward/List.vue`
- `resources/js/backend/views/product/components/ProductDetail.vue`
- `resources/js/backend/views/service-request/Detail.vue`
- `resources/js/backend/views/storage-contract/Detail.vue`
- `resources/js/backend/views/template/components/TemplateDetail.vue`
- `resources/js/frontend/components/VirtualTryon/PhotoSelector.vue`
- `resources/js/frontend/views/order/checkout.vue`
- `resources/js/frontend/views/product/detail.vue`
- `resources/js/frontend/views/register/index.vue`
- `resources/js/frontend/views/mypage/*`
- `resources/js/frontend/views/pages/*`

未追跡migration:

- `database/migrations/2026_05_16_000001_add_tryon_model_image_to_products_table.php`
- `database/migrations/2026_05_16_000002_update_about_us_company_info_page.php`
- `database/migrations/2026_05_17_000001_add_admin_note_to_storage_contracts_table.php`

これらはクライアント指示書の対応済み項目に対応する実装候補です。内容確認後、PR #13へ含めるか、別PRに分けるか判断してください。

## 特に確認してほしい未反映機能

以下はクライアント指示書上では対応済み扱いですが、develop/本番にはまだ入っていない可能性が高いです。

- 商品ごとのバーチャル試着用モデル写真 `tryon_model_image`
- バーチャル試着の顔写真条件説明
- メールテンプレートのプレビュー/テスト送信/変数確認
- 保管契約の担当者メモ `admin_note`
- レンタル注文時のカード有効期限チェック
- 注文詳細/CSVの備考、ヒップ、送料、裄削除など
- 保管契約一覧/詳細のID表記、リンク、課金補足
- 各種申請の確認チェック、文言、対象商品の保管ID表示
- オーナー報酬の振込先表示、CSV修正
- 会社情報ページの所在地/地図更新
- 未ログイン問い合わせ送信エラー修正
- 郵便番号からの住所自動入力安定化

## 仕様確認のまま残っている項目

以下4件は、クライアント確認または方針決定が必要です。

### `US-TOP-001` ロゴ

正しい差し替えロゴ素材・仕様が未確定です。  
本番反映予定のロゴがある場合、画像ファイルと表示サイズを確認してください。

### `MYPAGE-005` 退会手続きとユーザーデータ削除

ユーザー物理削除ではなく、退会状態化と個人情報匿名化を推奨しています。  
注文、決済、保管契約、報酬、問い合わせ履歴との整合性があるため、クライアントへ運用方針を確認してください。

### `ADMIN-USER-002` ユーザーデータ削除

`MYPAGE-005` と同じ論点です。  
管理画面に削除機能を作る場合も、物理削除ではなく匿名化機能として設計する方針が安全です。

### `MYPAGE-007` 収入とオーナーサービスの重複

以下の役割分離をクライアントへ提案する方針です。

- `収入`: オーナー報酬の確認専用
- `オーナーサービス`: 出品中商品の確認、出品申請、出品停止、利用停止、一時返却、マイタンス導線

この方針で問題ないか確認してください。

## 本番反映状況

バーチャル試着のコア機能は本番に入っています。

確認済み:

- 本番サーバ: `/var/www/html/moro`
- 本番ブランチ: `develop`
- 試着関連migration適用済み
  - `2026_04_11_000001_create_user_face_photos_table`
  - `2026_04_11_000002_create_virtual_tryon_results_table`
  - `2026_04_11_000003_add_tryon_consent_to_users_table`
- 本番 `.env`: `FACESWAP_DRIVER=replicate`

注意:

- `tryon_model_image` のmigrationは本番未適用です。
- `admin_note` のmigrationも本番未適用です。
- PR #11/#12/#13 はまだ未マージです。

## Liemへの依頼事項

1. PR #11/#12/#13を確認してください。
2. `client-feedback-20260428.md` の77件対応済み、4件仕様確認という整理が正しいか確認してください。
3. 未コミットの実装修正をレビューし、PR #13へ含めてよいか判断してください。
4. migration 3本の命名・内容・本番適用順に問題がないか確認してください。
5. PR #12のPAY.JP 3Dセキュアは本番疎通確認が必要か判断してください。
6. 仕様確認4件について、クライアントへそのまま確認してよいか確認してください。
7. マージ順は原則 `#11 -> #13 -> #12` を想定していますが、競合・依存関係があれば修正してください。

## 推奨マージ順

1. PR #11: 軽微修正
2. PR #13: クライアントフィードバック本体
3. PR #12: PAY.JP 3Dセキュア

理由:

- PR #11は小さいため先に入れやすい。
- PR #13は広範囲のUI/業務修正を含む想定。
- PR #12は決済影響が大きく、最後に単独確認する方が安全。

## 確認後にほしい返答

以下の形式で返答してください。

```text
PR #11: merge可 / 修正必要
PR #12: merge可 / 修正必要 / 本番疎通後
PR #13: docsのみでmerge / 実装修正を追加してmerge / PR分割

仕様確認4件:
- US-TOP-001:
- MYPAGE-005:
- MYPAGE-007:
- ADMIN-USER-002:

追加で修正が必要な項目:
- ...
```