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

受注CSVで、お届け時間IDが出力されない問題の修正 #5130

Merged
merged 1 commit into from
Aug 31, 2021

Conversation

matsuoshi
Copy link
Contributor

@matsuoshi matsuoshi commented Aug 31, 2021

概要(Overview・Refs Issue)

受注CSVで、お届け時間ID が常に出力されない問題に対応しました

コミュニティより
https://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=24235&forum=11&post_id=98760#forumpost98760

関連issue #4998

方針(Policy)

issue #4998 のとおりですが、dtb_csvのお届け時間IDの定義を修正しています。
また、送料IDは該当するカラムがないため、定義を削除しました。

実装に関する補足(Appendix)

csvの定義の修正・削除と、マイグレーションファイルの追加を行っています。

テスト(Test)

相談(Discussion)

レビュワー確認項目

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

@chihiro-adachi chihiro-adachi added this to the 4.1 milestone Aug 31, 2021
@codecov-commenter
Copy link

codecov-commenter commented Aug 31, 2021

Codecov Report

Merging #5130 (5f866e1) into 4.1 (ce55b43) will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##              4.1    #5130      +/-   ##
==========================================
+ Coverage   68.15%   68.17%   +0.02%     
==========================================
  Files         457      457              
  Lines       24988    24988              
==========================================
+ Hits        17030    17036       +6     
+ Misses       7958     7952       -6     
Flag Coverage Δ
tests 68.17% <ø> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...orage/Handler/SameSiteNoneCompatSessionHandler.php 91.04% <0.00%> (+2.98%) ⬆️
...ontroller/Admin/Setting/Shop/PaymentController.php 76.15% <0.00%> (+3.07%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ce55b43...5f866e1. Read the comment docs.

@chihiro-adachi
Copy link
Contributor

@matsuoshi
ありがとうございます。
新規インストールおよびマイグレーションで、以下確認しました。

  • お届け時間IDが追加されている
  • 送料IDが削除されている

コードも問題ないかと思います。

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

Successfully merging this pull request may close these issues.

3 participants