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.

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

対応状況管理機能の追加 #4844

Merged
merged 17 commits into from
Jan 26, 2021

Conversation

chihiro-adachi
Copy link
Contributor

@chihiro-adachi chihiro-adachi commented Jan 19, 2021

概要(Overview・Refs Issue)

#4842 対応状況管理機能の追加

方針(Policy)

受注ステータス設定をマスターデータから切り出し、ラベル等の編集が可能なUIを作成しています。

image

実装に関する補足(Appendix)

テスト(Test)

TODO

  • メッセージIDの修正
  • マスターデータでの受注ステータスの除外
  • テスト(UT、E2E)

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

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

レビュワー確認項目

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

@kiy0taka
Copy link
Contributor

色はカラーピッカー使えるといいかと思います。

@Yangsin
Copy link

Yangsin commented Jan 19, 2021

2.xにある対応状況管理と機能名がかぶるが、機能が違うので名称は見直したいです。
受注対応状況設定とかどうでしょうか。

@chihiro-adachi
Copy link
Contributor Author

@kiy0taka
ありがとうございます。
カラーピッカー対応しました。

image

@chihiro-adachi
Copy link
Contributor Author

@Yangsin
ありがとうございます。設定>店舗設定>受注対応状況設定にメニュー移動しました。

@chihiro-adachi chihiro-adachi marked this pull request as ready for review January 20, 2021 06:16
@chihiro-adachi chihiro-adachi changed the title [WIP] 対応状況管理機能の追加 対応状況管理機能の追加 Jan 20, 2021
@chihiro-adachi chihiro-adachi added the enhancement 機能追加 label Jan 20, 2021
@chihiro-adachi chihiro-adachi added this to the 4.0.6 milestone Jan 20, 2021
@okazy okazy self-assigned this Jan 26, 2021
@okazy
Copy link
Contributor

okazy commented Jan 26, 2021

動作確認して問題ありませんでした。
色変更のUIめっちゃ使いやすいです。

image

image

@okazy
Copy link
Contributor

okazy commented Jan 26, 2021

データ構造としては違和感がありますが、 mtb_customer_order_status も同様の画面で編集できるようにしてもいいかと思いました。

@chihiro-adachi
Copy link
Contributor Author

@okazy
確認ありがとうございます。

データ構造としては違和感がありますが、 mtb_customer_order_status も同様の画面で編集できるようにしてもいいかと思いました。

mtb_customer_order_statusすっかり忘れていました。確かにそうですね。追加しておきたいと思います。

@chihiro-adachi
Copy link
Contributor Author

@okazy もろもろ修正しました。
image

Copy link
Contributor

@okazy okazy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます!
動作、ソースコード共に問題ありません。

@chihiro-adachi
Copy link
Contributor Author

@okazy
確認ありがとうございます。

mtb_customer_order_statusはフロント側に表示される項目なのでツールチップで注意書きをつけておきました。

3cf66f2

@okazy
Copy link
Contributor

okazy commented Jan 26, 2021

テストが落ちていたので再実行しました。
ガチャだと思いますが近い機能だったため念の為貼り付けておきます。

image

@okazy okazy merged commit e20433c into EC-CUBE:4.0 Jan 26, 2021
@okazy
Copy link
Contributor

okazy commented Jan 26, 2021

ありがとうございます!マージしました!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 機能追加
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants