Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

複数出荷画面の実装 #3373

Merged
merged 46 commits into from
Jul 25, 2018
Merged

複数出荷画面の実装 #3373

merged 46 commits into from
Jul 25, 2018

Conversation

okazy
Copy link
Contributor

@okazy okazy commented Jul 20, 2018 •
edited
Loading

概要(Overview・Refs Issue)

  • 複数出荷登録画面の実装
  • 出荷管理の削除
  • 出荷管理で必要な処理は受注管理への移動

方針(Policy)

  • 受注管理と出荷管理が別れているとわかりにくいので受注の下に出荷が紐づく形に変更する
  • 単数出荷の場合
    • 出荷登録画面は不要で受注登録画面のみで確認・編集ができるようにする
    • 受注登録画面で商品の登録・削除が可能
  • 複数出荷の場合
    • 出荷登録画面で出荷内容の確認・編集ができるようにする
    • 出荷登録画面では数量の変更は可能・価格の変更は不可
    • 受注登録画面で商品の登録・削除は不可、その他明細の追加削除は可能

実装に関する補足(Appendix)

  • 出荷登録画面では受注一覧画面での出荷済み処理・出荷完了メール送信処理を流用している
    • 受注一覧画面と出荷登録画面で共通の処理(js部分)は別のtwigファイルに切り出している
  • 配送先情報のフォームは受注登録画面のフォームと合わせた

テスト（Test)

  • 基本的な操作のテストケースを追加

相談（Discussion）

@okazy okazy changed the title 複数出荷画面の実装 [WIP]複数出荷画面の実装 Jul 20, 2018
@ryo-endo ryo-endo added this to the 3.n.0 milestone Jul 21, 2018
okazy and others added 18 commits July 23, 2018 09:44
@okazy
Merge remote-tracking branch 'chihiro-adachi/improve/akaden' into fea…
3bfcebd 
…t-admin-multi-shipping
@okazy
feat 復習配送管理 商品追加時にIDを指定しないように修正
16ee5f3
@okazy
[WIP] feat 復習配送管理 商品追加と削除
0917a94
@okazy
feat 復習配送管理 不要になったルーティングを削除
bb26d82
@nanasess
Renamed
03d950d
@nanasess
送り状番号登録のテスト追加
49df201
@okazy
feat 復習配送管理 お届け先を追加できるように修正
db0dae1 
TODO: 機能していないコードを一旦コメントアウト
@okazy
Merge remote-tracking branch 'chihiro-adachi/improve/akaden' into fea…
03aaa9c 
…t-admin-multi-shipping

# Conflicts:
#	src/Eccube/Resource/template/admin/Shipping/edit.twig
#	tests/Eccube/Tests/Web/Admin/Order/ShippingEditControllerTest.php
@okazy
fix 受注編集画面 お届け時間が更新されない不具合の修正
2bdf000
@okazy
fix cs-fixer
4fff61c
@okazy
fix cs-fixer
6659046
@nanasess
出荷情報編集のテスト修正
18315b4
@nanasess
Merge remote-tracking branch 'okazy/feat-admin-multi-shipping' into o…
b553d58 
…kazy-feat-admin-multi-shipping
@okazy
fix 複数配送画面 配送業者変更で配送時間が変わるように修正
c6731d8
@okazy
Merge pull request #17 from nanasess/okazy-feat-admin-multi-shipping
48f8970 
正常系のテスト追加
@okazy
Merge remote-tracking branch 'origin/feat-admin-multi-shipping' into …
e25e6e6 
…feat-admin-multi-shipping
@okazy
fix cs-fixer
7befcb6
@okazy
fix 出荷CSV URLの変更に対応
306b5e0
okazy added 10 commits July 24, 2018 11:14
@okazy
feat 複数配送画面 出荷済み処理を追加
f971303
@okazy
Merge remote-tracking branch 'upstream/experimental/sf' into feat-adm…
198f0c1 
…in-multi-shipping

# Conflicts:
#	src/Eccube/Controller/Admin/Order/OrderController.php
#	tests/Eccube/Tests/Web/Admin/Order/CsvImportControllerTest.php
@okazy
feat 複数配送画面 出荷完了メール送信ボタンの実装
dd86746
@okazy
feat 複数配送画面 出荷完了ボタンの実装、コンフリクト改称時にデグレしていた部分を修正
a651864
@okazy
refactor 受注一覧画面
152ad10
@okazy
refactor 複数配送画面 不要なuseを削除
36f23bb
@okazy
Merge remote-tracking branch 'upstream/experimental/sf' into feat-adm…
2061933 
…in-multi-shipping

# Conflicts:
#	src/Eccube/Controller/Admin/Order/OrderController.php
@okazy
TODO 出荷に出荷日を入れる条件の見直しが必要
cd0eef9
@okazy
fix 受注編集画面 複数配送の場合はお届け先フォームが出力されないため郵便番号のライブラリでエラーが発生したのを修正
48203a4
@okazy
fix 受注一覧画面 複数配送管理へのリンクを修正、未出荷の場合も複数出荷管理へのリンクが出力されるように修正
f0418c5
@okazy okazy changed the title [WIP]複数出荷画面の実装 複数出荷画面の実装 Jul 24, 2018
nanasess and others added 8 commits July 24, 2018 18:21
@nanasess
お届け先追加のテスト追加
ec118e7
@okazy
feat 出荷管理自体がなくなったので不要なファイルを削除
142d71e
@okazy
Merge pull request #18 from nanasess/okazy-feat-admin-multi-shipping
0298ad6 
お届け先追加のテスト追加
@okazy
feat 複数出荷管理画面 出荷完了・出荷完了メール送信時に文言を追加
7d22ae8
@okazy
Merge remote-tracking branch 'origin/feat-admin-multi-shipping' into …
0a530f5 
…feat-admin-multi-shipping
@okazy
feat 複数出荷管理画面 ページタイトルの修正
71d0cae
@okazy
feat 複数出荷管理画面 ページタイトルの修正・不要なテストファイルを削除
b74b28d
@okazy
Merge remote-tracking branch 'upstream/experimental/sf' into feat-adm…
497efcb 
…in-multi-shipping

# Conflicts:
#	src/Eccube/Resource/template/admin/Order/index.twig
@chihiro-adachi chihiro-adachi merged commit 4304926 into EC-CUBE:experimental/sf Jul 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
4.0
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@okazy @chihiro-adachi @ryo-endo @nanasess