検索結果一覧を非React化した by nakamu-kazu222 · Pull Request #8801 · fjordllc/bootcamp

nakamu-kazu222 · 2025-06-15T06:20:22Z

本PRは、#8203 の実装を移行したものです。
#8203 で多数のコンフリクトが発生したので、本PRおよびブランチを再作成しました。

Issue

検索結果一覧を非React化したい #8040

概要

変更確認方法

1. 環境構築

feature/change-searchables-not-reactをローカルに取り込む
rails db:resetを実行する
bin/setupを実行する
foreman start -f Procfile.devを実行する
適当なユーザーでログインする(以下の手順の画像では、machidaユーザーでログインしている)

2. 検索の確認

検索ワードなしでの検索

右上の検索バーで「すべて」を選び、検索ワードはなしで検索する

すべての検索結果が表示されることを確認する

検索結果は、「提出物などの検索結果の種類」、「検索結果のタイトル(リンク付き)」、「検索結果のDescription」、「ユーザーアイコン、ユーザー名(リンク付き)」、「更新時間」が表示されていること

コメントの場合は、タイトルの横に「コメント」と表示されていること

WIPの場合は、タイトルの横に「WIP」と表示されていること

右上の検索バーで「お知らせ」を選び、検索ワードはなしで検索する

お知らせのみ検索結果が表示されることを確認する
コメントの場合は、「更新時間」の右に「お知らせを作成したユーザー名(リンク付き)お知らせ」が表示されていること

右上の検索バーで「プラクティス」を選び、検索ワードはなしで検索する

プラクティスのみの検索結果が表示されることを確認する

右上の検索バーで「日報」を選び、検索ワードはなしで検索する

日報のみ検索結果が表示されることを確認する
コメントの場合は、「更新時間」の右に「日報を作成したユーザー名(リンク付き)日報」が表示されていること

右上の検索バーで「提出物」を選び、検索ワードはなしで検索する

提出物のみ検索結果が表示されることを確認する
コメントの場合は、「更新時間」の右に「提出物を作成したユーザー名(リンク付き)提出物」が表示されていること

右上の検索バーで「Q&A」を選び、検索ワードはなしで検索する

Q&Aのみ検索結果が表示されることを確認する
コメントの場合は、「更新時間」の右に「Q&Aを作成したユーザー名(リンク付き)Q&A」が表示されていること

右上の検索バーで「Docs」を選び、検索ワードはなしで検索する

Docsのみ検索結果が表示されることを確認する
コメントの場合は、「更新時間」の右に「Docsを作成したユーザー名(リンク付き)Docs」が表示されていること

右上の検索バーで「イベント」を選び、検索ワードはなしで検索する

イベントのみ検索結果が表示されることを確認する
コメントの場合は、「更新時間」の右に「イベントを作成したユーザー名(リンク付き)イベント」が表示されていること

右上の検索バーで「定期イベント」を選び、検索ワードはなしで検索する

定期イベントのみ検索結果が表示されることを確認する
コメントの場合は、「更新時間」の右に「定期イベントを作成したユーザー名(リンク付き)定期イベント」が表示されていること

右上の検索バーで「ユーザー」を選び、検索ワードはなしで検索する

ユーザーのみ検索結果が表示されることを確認する

検索ワードありでの検索

「すべて」を選び、1語の単語で検索する

検索した1語の単語が含まれる検索結果が表示されること
検索ワードが太字で表示されること

「すべて」を選び、2語以上の単語で検索する

検索した2語以上の単語が含まれる検索結果が表示されること
検索ワードが太字で表示されること

「ユーザー」を選び、ユーザー名で検索する

検索したユーザーが表示されること
ログインユーザーが、メンター権限がある場合にユーザーを検索した時に、「更新時間」の右に「相談部屋(リンク付き)」が表示されること

ログインユーザーが、メンター権限がない場合にユーザーを検索した時に、「更新時間」の右に何も表示されないこと

検索したい項目を選び、「user:ユーザー名」で検索する

検索したユーザーが更新した検索結果が表示されること

Screenshot

非React化のみなので、画面の変更はなし

Summary by CodeRabbit

新機能
- 検索機能を全面的に刷新し、複数キーワードやユーザー指定による絞り込み、権限に応じた検索結果の表示に対応しました。
- 記事の「注目の記事」タグによる特集記事表示スコープを追加し、トップページなどで動的に表示されるようになりました。
- 検索結果を表現する新クラスを導入し、検索結果の情報を整理・強化しました。
改善・変更
- 検索結果ページをサーバーサイドレンダリングに変更し、バッジやユーザー情報、管理者向けリンクなどを詳細に表示。
- 記事投稿フォームのタグ入力ヘルプを拡充し、各ページへの掲載方法を明記。
- マークダウンからプレーンテキストへの変換やHTMLサニタイズ処理を統一化し、新たなユーティリティメソッドを追加。
- ユーザーページのタブ表示条件を環境や権限に応じて制御。
- 検索ヘルパーのURL生成や表示ロジックを整理し、アクセス制御を強化。
- タグやフィクスチャデータ、テストデータを「注目の記事」対応で拡充・修正。
- 検索処理の内部ロジックを分割・整理し、ユーザー視点の絞り込みや複数キーワード対応を実装。
- コントローラーの検索処理を充実化し、API側の検索エンドポイントを廃止。
- 検索関連のReactコンポーネントを削除し、サーバーサイドでのレンダリングに統一。
- 検索対象モデルに検索ヘルパーを追加し、検索関連の振る舞いを統一。
バグ修正
- 一部コントローラーやヘルパーのタイプミス修正・条件分岐の見直し。
スタイル
- 特集記事の説明文に新しいCSSスタイルを追加。
テスト
- 検索・記事・コメント・マークダウン処理・特集記事表示に関するテストを追加・拡充。
- 検索ヘルパーのURL生成や表示制御に関するテストを強化。
- マークダウン変換のユーティリティに対する単体テストを追加。
- API検索エンドポイント関連の統合テストを削除。
- システムテストのセレクター修正や表示内容の微調整を実施。
その他
- 不要となったAPIエンドポイントやReactコンポーネント、関連テストの削除。
- スキーマの細かな調整やインデックスの整理を実施。

undefined method `strip_tags'エラーがでたため strip_tags を使うようにするためActionView::Helpers::SanitizeHelperを利用したまた、明示的に ActionController::Base.helpers.strip_tags を使うように変更した

rails test test/system/pages_test.rb:296が落ちていた

result_forと同じようなメソッド名にした

markdown_to_plain_textでHTMLに変換した影響があるテストエラーを解消した rails test test/system/current_user/watches_test.rb:53 rails test test/system/current_user/bookmarks_test.rb:25

markdown_to_plain_textでHTMLに変換した影響があるテストエラーを解消した

API::SearchablesTestは不要なので削除した SearchableTestでテスト結果数が異なったので修正した

SearchResultでstrip_tagsを使用するために、strip_tags をActionView::Base.full_sanitizer.sanitize に置き換えた

テストが落ちていたため修正した

該当プラクティスを修了するまで他の人の提出物へのコメントは見れないようにするテストの修正

test/fixtures/comments.ymlのデータを追加したので、テスト結果を変更した

markdown_to_plain_textでHTMLに変換する部分を修正したので、元に戻した

サマリーのメソッドを変更した

coderabbitaiから、パフォーマンスの問題が発生する可能性があるとのことで、リファクタリングした

nakamu-kazu222 · 2025-07-04T14:10:39Z

本PRで、コンフリクトが多数発生してしまったので、#8887 を再作成しました
そのため、本PRはCloseしまして、#8887 で実装をさせていただきます

nakamu-kazu222 · 2025-07-04T14:11:16Z

feature/change-searchables-not-react2ブランチを削除しました

umizaru and others added 30 commits June 11, 2025 16:12

特定のタグを持つ最新のブログ記事6件を表示

b285e69

fixturesを追加

7be41a7

Homeコントローラーにメソッドを追加、テストを修正

3e4c062

注目のブログ記事のレイアウトを整えた

ac7df6a

コンフリクトを解消

7c6ed28

注目の記事のデザインを整えた。要約がない場合、本文の最初の100文字をHTMLタグを省いて表示するようにした。

7455667

不要な分岐を削除

65712a0

ホーム画面表示用のタグを「feature」から「注目の記事」に変更

9594696

ブログ記事一覧に注目の記事と表示させる条件を変更した

630dae2

タグの登録時のヘルプ文章を追加

3c68ae8

test/helpers/articles_helper_test.rbを修正

5be8e74

articles.yml の日付計算を each_with_index に変更

ea751fe

YAML name のコロンを削除

8de5f0e

「注目の記事」取得処理をスコープに統一して重複コードを削除

857ec2b

YAML を修正

a69afa3

ループでfeatureタグを生成するように修正

a5ba40a

注目の記事表示順序を検証、fixture追加

0cec80b

fixtureの tag_listを削除、「注目の記事」タグをsetupで付与

2f5429d

MarkdownHelperのテストを追加

b07041d

RedcarpetからCommonMarkerに置き換え、scriptタグ除去するように修正

0a01dc9

メンター取得処理を元の実装に戻した

340cdde

rubocopに対する修正

bfd4f2f

featureに記事のfeatureタグを追加

3c5b528

feature タグ定義（注目の記事）を追加

b88b774

テストファイルもループでfeatureタグを生成するように修正

83f2e92

setupを削除

3ace10c

不要なgemを削除

2a571e5

markdown_to_plain_text メソッドの処理を簡略化、テストの修正

466eb73

Article.featuredスコープのテスト追加

a5750ca

テストファイルもループでfeatureタグを生成するように修正

9d703ce

nakamu-kazu222 added 20 commits July 3, 2025 20:40

includeの位置を変更した

37a1955

rubocopを実行した

d4f9f71

テストが落ちていたため、メソッドを追加した

e86b953

rails test test/system/pages_test.rb:296が落ちていた

Rubcopを実行した

234e956

不要なapi/searchables#indexを消去した

f1db714

メソッド名を変更した

46599d1

result_forと同じようなメソッド名にした

rebaseしてmarkdown_to_plain_textを使用できるようにした

07917b6

テストエラーを解消した

8d6f304

markdown_to_plain_textでHTMLに変換した影響があるテストエラーを解消した rails test test/system/current_user/watches_test.rb:53 rails test test/system/current_user/bookmarks_test.rb:25

テストエラーを解消した

bcf4166

markdown_to_plain_textでHTMLに変換した影響があるテストエラーを解消した

テストを修正した

ed2d9bc

API::SearchablesTestは不要なので削除した SearchableTestでテスト結果数が異なったので修正した

Commonmarkerを使わない実装にした

3e43f60

SearchResultでstrip_tagsを使用するために、strip_tags をActionView::Base.full_sanitizer.sanitize に置き換えた

テストを修正した

988fbd6

テストが落ちていたため修正した

テストを修正した

978061f

該当プラクティスを修了するまで他の人の提出物へのコメントは見れないようにするテストの修正

watchableのサマリーがbodyなため修正した

08dbd2d

テストを修正した

e49c726

test/fixtures/comments.ymlのデータを追加したので、テスト結果を変更した

修正したテストを元に戻した

55bbad8

markdown_to_plain_textでHTMLに変換する部分を修正したので、元に戻した

テストデータで重複している部分があったので、修正した

04e131f

テストが落ちていたため修正した

3a8127b

サマリーのメソッドを変更した

test/fixtures/comments.ymlを整理したので、影響があるテストを修正した

a13ee2a

nakamu-kazu222 force-pushed the feature/change-searchables-not-react2 branch from 0651451 to a13ee2a Compare July 3, 2025 11:41

kitarou888 mentioned this pull request Jul 3, 2025

検索機能を拡充：自分のみのチェックボックスを追加 #8277

Merged

nakamu-kazu222 added 3 commits July 4, 2025 17:15

リファクタリングした

db47f3c

coderabbitaiから、パフォーマンスの問題が発生する可能性があるとのことで、リファクタリングした

コンフリクトを解消した

df2f0fe

テストが落ちていたため修正した

6e474b4

nakamu-kazu222 mentioned this pull request Jul 4, 2025

検索結果一覧を非React化した #8887

Merged

nakamu-kazu222 closed this Jul 4, 2025

nakamu-kazu222 deleted the feature/change-searchables-not-react2 branch July 4, 2025 14:10

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

検索結果一覧を非React化した#8801

検索結果一覧を非React化した#8801
nakamu-kazu222 wants to merge 228 commits intomainfrom
feature/change-searchables-not-react2

nakamu-kazu222 commented Jun 15, 2025 •

edited by coderabbitai bot

Loading

Uh oh!

nakamu-kazu222 commented Jul 4, 2025

Uh oh!

nakamu-kazu222 commented Jul 4, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

17 participants

Conversation

nakamu-kazu222 commented Jun 15, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Issue

概要

変更確認方法

1. 環境構築

2. 検索の確認

検索ワードなしでの検索

検索ワードありでの検索

Screenshot

Summary by CodeRabbit

Uh oh!

nakamu-kazu222 commented Jul 4, 2025

Uh oh!

nakamu-kazu222 commented Jul 4, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

17 participants

nakamu-kazu222 commented Jun 15, 2025 •

edited by coderabbitai bot

Loading