管理機能-受注一覧 : CSV出力を行っても「出荷ID」が見つからないがおそらく「配送ID」がそれであるためCSVヘッダータイトルデータを修正.

[genzouw]

概要(Overview・Refs Issue)

管理機能-受注一覧 : CSV出力を行っても「出荷ID」が見つからないがおそらく「配送ID」がそれであるためCSVヘッダータイトルデータを修正しました。

方針(Policy)

• 出荷CSV登録* ページにならい、 「出荷ID」 でシステム全体を統一したほうが良いかと思いました。

実装に関する補足(Appendix)

テスト（Test)

ローカル環境で動作確認済み。

相談（Discussion）

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更
• フックポイントの呼び出しタイミングの変更
• フックポイントのパラメータの削除・データ型の変更
• twigファイルに渡しているパラメータの削除・データ型の変更
• Serviceクラスの公開関数の、引数の削除・データ型の変更
• 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[genzouw]

※ちなみに相談ですが、「配送」の文言については「出荷」で統一したりする必要はありませんか？

---

簡単に調査した結果、それぞれの文言は以下のような出現傾向となっていました。

$ ag 配送先 . | wc -l
50
$ ag 出荷先 . | wc -l
11
$ ag 出荷 . | wc -l
174
$ ag 配送 . | wc -l
224
$ ag 発送 . | wc -l
34

[okazy]

ありがとうございます、文言は「出荷」に統一したいですね。
初期データの更新ですが、マイグレーションまでは不要かと思います。

[chihiro-adachi]

@genzouw @okazy

ちなみに相談ですが、「配送」の文言については「出荷」で統一したりする必要はありませんか

フロントと管理画面で用語が変わる場合があるので、そのあたりも精査したいですね。