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.1-feature ブランチを 4.1-beta2 ブランチに取り込み #5000

Merged
merged 328 commits into from
Apr 5, 2021

Conversation

nanasess
Copy link
Contributor

@nanasess nanasess commented Apr 5, 2021

概要(Overview・Refs Issue)

4.1 beta2 のリリースに向けて4.1-featureブランチの修正を4.1-beta2ブランチに取り込み

refs #4901 (comment)

方針(Policy)

  • 4.1-featureブランチの修正を4.1-beta2ブランチにマージ
  • php-cs-fixer の実行

実装に関する補足(Appendix)

  • Symfony 4.4.18 から 4.4.21 の変更でテストが大量に落ちるため、今回は保留。別途 PR します

テスト(Test)

GitHub Actions が正常に通るのを確認

相談(Discussion)

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

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

レビュワー確認項目

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

oywc410 and others added 30 commits December 1, 2020 09:12
特定のPDFテンプレートで表が正常に出力されない不具合を修正
ページ管理の編集対象に会員登録(確認ページ)を加える EC-CUBE#4428
カートボタンにアイコンなどhtmlを追加している場合に、規格選択によってそれが消えてしまう不具合の対応
機種依存文字が文字化けする不具合を修正
CSVで商品規格を登録時に任意の項目がない場合のエラーを修正
Bumps [ini](https://github.com/isaacs/ini) from 1.3.5 to 1.3.7.
- [Release notes](https://github.com/isaacs/ini/releases)
- [Commits](npm/ini@v1.3.5...v1.3.7)

Signed-off-by: dependabot[bot] <[email protected]>
okazy and others added 17 commits March 25, 2021 16:16
codeception.sh を Travis CI 連携廃止に伴い削除
IP制限(拒否リスト)とログイン履歴とE2Eテストを追加
…tory

管理画面ログイン履歴とIPアドレスによるアクセス拒否を実装
nginx.conf.sample に bin ディレクトリを追加
docker-compose ファイルを分割する
管理画面のお知らせのE2Eテストをリダイレクトに対応
…y18n-3.2.2

Bump y18n from 3.2.1 to 3.2.2
4.0ブランチの修正を4.1-featureブランチに取り込み
@codecov-io
Copy link

Codecov Report

Merging #5000 (9101b77) into 4.0 (23da54c) will decrease coverage by 0.17%.
The diff coverage is 80.82%.

Impacted file tree graph

@@             Coverage Diff              @@
##                4.0    #5000      +/-   ##
============================================
- Coverage     76.35%   76.17%   -0.18%     
+ Complexity     6189     5922     -267     
============================================
  Files           445      446       +1     
  Lines         20994    20831     -163     
============================================
- Hits          16029    15869     -160     
+ Misses         4965     4962       -3     
Flag Coverage Δ Complexity Δ
tests 76.17% <80.82%> (-0.18%) 0.00 <330.00> (ø)

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

Impacted Files Coverage Δ Complexity Δ
src/Eccube/Controller/Admin/AdminController.php 78.63% <ø> (+0.45%) 28.00 <0.00> (ø)
...cube/Controller/Admin/Content/LayoutController.php 73.86% <ø> (ø) 22.00 <0.00> (ø)
...Controller/Admin/Content/MaintenanceController.php 0.00% <ø> (ø) 8.00 <0.00> (ø)
...cube/Controller/Admin/Order/ShippingController.php 87.25% <ø> (-0.13%) 29.00 <0.00> (-2.00)
...ontroller/Admin/Product/ProductClassController.php 84.74% <ø> (-0.09%) 55.00 <0.00> (-1.00)
...ntroller/Admin/Setting/Shop/DeliveryController.php 84.83% <ø> (-0.42%) 41.00 <0.00> (-3.00)
...e/Controller/Admin/Setting/Shop/MailController.php 86.88% <ø> (ø) 13.00 <0.00> (ø)
.../Controller/Admin/Setting/System/LogController.php 61.36% <0.00%> (+1.36%) 11.00 <0.00> (-3.00) ⬆️
...ller/Admin/Setting/System/MasterdataController.php 95.45% <ø> (-0.06%) 18.00 <0.00> (-1.00)
...ntroller/Admin/Setting/System/MemberController.php 85.58% <ø> (ø) 17.00 <0.00> (ø)
... and 269 more

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 23da54c...9101b77. Read the comment docs.

@okazy okazy added the improvement 機能改善 label Apr 5, 2021
@okazy okazy added this to the 4.1 milestone Apr 5, 2021
@okazy okazy changed the base branch from 4.0 to 4.1-beta2 April 5, 2021 04:51
@okazy
Copy link
Contributor

okazy commented Apr 5, 2021

プルリク先のブランチを 4.0 -> 4.1-beta2 に切り替えました

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 4574bdd into EC-CUBE:4.1-beta2 Apr 5, 2021
@okazy
Copy link
Contributor

okazy commented Apr 5, 2021

ありがとうございます!テストが通っていましたのでマージしました!

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

Successfully merging this pull request may close these issues.