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

お知らせ変更に対応 #4046

Merged

1 commit merged into EC-CUBE:4.0 from okazy:fix-test-for-news

Dec 12, 2018

Contributor

okazy commented Dec 12, 2018

概要(Overview・Refs Issue)

管理画面ホームのiFrameで取得しているお知らせの内容変更に伴いE2Eテストが落ちてしまっていたので修正

方針(Policy)

本当はiFrameの情報に依存しない実装としたいが、いったん最小限の変更にとどめた

実装に関する補足(Appendix)

テスト（Test)

相談（Discussion）

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

なし

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


          fix お知らせ変更に対応

1a4c364

okazy added this to the 4.0.2 milestone

ghost merged commit 6693703 into EC-CUBE:4.0

This pull request was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet