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

[4.2]admin の bootstrap5対応 #5446

Merged
merged 15 commits into from
Jul 21, 2022

Conversation

tsumu-tsumu
Copy link
Contributor

@tsumu-tsumu tsumu-tsumu commented Jul 15, 2022

概要(Overview・Refs Issue)

 - adminのbootstrap5対応です。
bootstrap5に合わせたclass名の変更やbootstrap_4_layout.html.twigの修正、scssを修正しております。

方針(Policy)

src/Eccube/Resource/template/admin/Form/bootstrap_4_layout.html.twig
src/Eccube/Resource/template/admin/Form/bootstrap_4_horizontal_layout.html.twig
11行目に{%- extends 'bootstrap_5_horizontal_layout.html.twig' -%}、{%- extends 'bootstrap_5_layout.html.twig' -%}を追加しており、既存の仕様に影響がない範囲でbootstrap5のマークアップを読み込んでいます。

また日付まわりはDatepickerに修正しました

実装に関する補足(Appendix)

編集ファイルが多かったため、問題ないかご確認お願いいたします。

テスト(Test)

これからすべての箇所をもう一度チェックします。
チェック後、結果をお伝えいたします。

相談(Discussion)

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

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

レビュワー確認項目

  • 動作確認
  • コードレビュー
  • E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • 互換性が保持されているか
  • セキュリティ上の問題がないか
    • 権限を超えた操作が可能にならないか
    • 不要なファイルアップロードがないか
    • 外部へ公開されるファイルや機能の追加ではないか
    • テンプレートでのエスケープ漏れがないか

@tsumu-tsumu tsumu-tsumu changed the base branch from 4.1 to 4.2 July 15, 2022 08:42
@nanasess nanasess changed the title 202200708 admin bootstrap5 [4.2]admin の bootstrap5対応 Jul 15, 2022
@tsumu-tsumu
Copy link
Contributor Author

もう一度、対象箇所をすべて確認し、修正を加えました。
datepicker実装予定の箇所を覗いて、問題ないかと思います。

@chihiro-adachi chihiro-adachi added this to the 4.2.0 milestone Jul 19, 2022
@nanasess nanasess force-pushed the 202200708_admin_bootstrap5 branch 2 times, most recently from 0bc2a5b to e924400 Compare July 20, 2022 01:08
@nanasess nanasess force-pushed the 202200708_admin_bootstrap5 branch from e924400 to 1d04cd9 Compare July 20, 2022 05:22
@nanasess nanasess marked this pull request as ready for review July 20, 2022 05:54
@chihiro-adachi
Copy link
Contributor

@nanasess @tsumu-tsumu
こちら修正は完了でよいでしょうか?

@nanasess
Copy link
Contributor

@chihiro-adachi はい、大丈夫です

@chihiro-adachi chihiro-adachi merged commit 8a2cd75 into EC-CUBE:4.2 Jul 21, 2022
@chihiro-adachi
Copy link
Contributor

@nanasess @tsumu-tsumu
ありがとうございます。マージしました。

@chihiro-adachi
Copy link
Contributor

#5409

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

Successfully merging this pull request may close these issues.

3 participants