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

「※ は軽減税率対象商品です。」表示判断を追加 #4438

Merged
merged 2 commits into from
Feb 5, 2020

Conversation

oywc410
Copy link
Contributor

@oywc410 oywc410 commented Jan 14, 2020

概要(Overview・Refs Issue)

#4304

軽減税率商品存在しない場合は「※ は軽減税率対象商品です。」が非表示にする

##対象範囲

  • ご注文手続き画面
  • ご注文内容のご確認画面
  • マイページの注文履歴画面
  • 購入完了時のメール
  • 納品書を出力

@okazy okazy added affected:plugin プラグインに影響する可能性のある実装変更 improvement 機能改善 Status: needs-review labels Jan 17, 2020
@okazy okazy added this to the 4.0.4 milestone Jan 17, 2020
@okazy
Copy link
Contributor

okazy commented Feb 5, 2020

購入完了時のテキストメールの改行が軽減税率対象外の場合に崩れてしまっていたので、うまく表示されるように修正しました。

変更前

スクリーンショット 2020-02-05 14 23 14

変更後

スクリーンショット 2020-02-05 14 25 35

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.

変更内容、動作確認して問題ありませんでした。

@okazy okazy merged commit 425a1d9 into EC-CUBE:4.0 Feb 5, 2020
@okazy
Copy link
Contributor

okazy commented Feb 5, 2020

取り込みました、ありがとうございます。

@okazy okazy added affected:template フロントテンプレートの変更 and removed affected:plugin プラグインに影響する可能性のある実装変更 labels Jun 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affected:template フロントテンプレートの変更 improvement 機能改善 Status: ready-for-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants