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

管理画面 商品のタグによる絞り込み #4975

Merged
merged 1 commit into from
Nov 25, 2021

Conversation

matsuoshi
Copy link
Contributor

@matsuoshi matsuoshi commented Mar 18, 2021

概要(Overview・Refs Issue)

4.1-feature 向けのPRです。
管理画面の商品一覧にて、タグによる商品の絞り込みを行えるようにします。
#4862

スクリーンショット 2021-03-18 15 36 27

方針(Policy)

ひとまず #4862 タグ検索機能に関し、管理画面部分のみのPRを作成しました。

フロント画面の検索については別途作成 #4976

実装に関する補足(Appendix)

テスト(Test)

タグ単体の絞り込みと、タグ・商品名のAND検索について、
テストコードを記述しています。

相談(Discussion)

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

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

レビュワー確認項目

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

新機能追加に伴う競合確認

@chihiro-adachi chihiro-adachi changed the base branch from 4.1-feature to 4.1 September 6, 2021 05:43
@codecov-commenter
Copy link

Codecov Report

Merging #4975 (9db51c2) into 4.1 (5691c3b) will decrease coverage by 0.02%.
The diff coverage is 42.10%.

Impacted file tree graph

@@            Coverage Diff             @@
##              4.1    #4975      +/-   ##
==========================================
- Coverage   68.23%   68.21%   -0.03%     
==========================================
  Files         455      455              
  Lines       24930    24948      +18     
==========================================
+ Hits        17012    17018       +6     
- Misses       7918     7930      +12     
Flag Coverage Δ
tests 68.21% <42.10%> (-0.03%) ⬇️

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

Impacted Files Coverage Δ
src/Eccube/Form/Type/Admin/SearchProductType.php 58.33% <26.66%> (-3.64%) ⬇️
src/Eccube/Repository/ProductRepository.php 97.00% <100.00%> (+0.07%) ⬆️
src/Eccube/Entity/TaxRule.php 85.18% <0.00%> (-1.24%) ⬇️

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 5691c3b...9db51c2. Read the comment docs.

@nanasess
Copy link
Contributor

nanasess commented Sep 6, 2021

タグは1商品に複数設定される場合が多いので、複数選択できると嬉しいですね

@matsuoshi matsuoshi added this to no status in 4.1.1 Nov 15, 2021
@matsuoshi matsuoshi modified the milestones: 4.1.x, 4.1.1 Nov 15, 2021
@matsuoshi matsuoshi moved this from no status to To Do in 4.1.1 Nov 15, 2021
@nanasess nanasess moved this from To Do to Ready for Merge in 4.1.1 Nov 24, 2021
@carkn
Copy link
Contributor

carkn commented Nov 25, 2021

本件、マージさせていただきます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
4.1.1
Done
Development

Successfully merging this pull request may close these issues.

None yet

6 participants