Skip to content

[サイト内検索] 隠しページ(ページ管理>メニュー非表示のページ)は検索に含めない設定を追加#2386

Merged
masaton0216 merged 10 commits intoopensource-workshop:masterfrom
akagane99:connect-cms-ideas/issues/183
Apr 13, 2026
Merged

[サイト内検索] 隠しページ(ページ管理>メニュー非表示のページ)は検索に含めない設定を追加#2386
masaton0216 merged 10 commits intoopensource-workshop:masterfrom
akagane99:connect-cms-ideas/issues/183

Conversation

@akagane99
Copy link
Copy Markdown
Contributor

概要

修正後画面

サイト内検索>設定変更(新規作成)

image

レビュー完了希望日

急ぎません

関連Pull requests/Issues

なし

参考

なし

DB変更の有無

あり

チェックリスト

@masaton0216 masaton0216 self-requested a review March 25, 2026 01:47
@masaton0216 masaton0216 self-assigned this Mar 25, 2026
@masaton0216 masaton0216 added the enhancement 機能強化 label Mar 25, 2026
Copy link
Copy Markdown
Contributor

@masaton0216 masaton0216 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご対応ありがとうございます。
下記、ご確認お願いいたします。

  • パフォーマンス観点で2つ
  • enum化の検討依頼1つ
  • スペース続きの軽微な修正1つ

Comment thread app/Plugins/User/Searchs/SearchsPlugin.php Outdated
Comment thread app/Plugins/User/Searchs/SearchsPlugin.php Outdated
Comment thread app/Plugins/User/Searchs/SearchsPlugin.php Outdated
Comment thread app/Plugins/User/Searchs/SearchsPlugin.php Outdated
@masaton0216 masaton0216 assigned akagane99 and unassigned masaton0216 Mar 25, 2026
@akagane99
Copy link
Copy Markdown
Contributor Author

@masaton0216 さん
指摘事項を修正しました。お時間あるときに、再レビューをお願いします。
(ついでで フレームの選択のenum化 も対応もさせて頂きました。)

Copy link
Copy Markdown
Contributor

@masaton0216 masaton0216 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご対応ありがとうございます。
私のミスリーディングで二度手間になってしまいすいません。
追加したenumクラスの部分で4カ所、他と合わせて欲しい箇所があります。

Comment thread app/Enums/SearchsFrameSelect.php Outdated
Comment thread app/Enums/SearchsPageSelect.php Outdated
Comment thread app/Enums/SearchsFrameSelect.php Outdated
Comment thread app/Enums/SearchsPageSelect.php Outdated
@akagane99
Copy link
Copy Markdown
Contributor Author

@masaton0216 さん
enumsの指摘事項を修正しました。お時間あるときに、再レビューをお願いします。

Copy link
Copy Markdown
Contributor

@masaton0216 masaton0216 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご対応ありがとうございます。LGTMです。

@masaton0216 masaton0216 merged commit 0b0054e into opensource-workshop:master Apr 13, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement 機能強化

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants