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

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

Draft
wants to merge 61 commits into
base: main
Choose a base branch
from

Conversation

nakamu-kazu222
Copy link
Contributor

Issue

概要

変更確認方法

  1. {branch_name}をローカルに取り込む

Screenshot

変更前

変更後

@nakamu-kazu222 nakamu-kazu222 self-assigned this Nov 18, 2024
@nakamu-kazu222 nakamu-kazu222 force-pushed the feature/change-searchables-not-react branch from 71d671b to 542291e Compare November 27, 2024 04:00
@nakamu-kazu222 nakamu-kazu222 force-pushed the feature/change-searchables-not-react branch from d5c474d to d33ddcd Compare December 21, 2024 18:50
@nakamu-kazu222 nakamu-kazu222 force-pushed the feature/change-searchables-not-react branch from d33ddcd to 3d3e414 Compare January 5, 2025 01:40
@nakamu-kazu222 nakamu-kazu222 force-pushed the feature/change-searchables-not-react branch from f79c125 to 13d7d09 Compare February 1, 2025 06:36
@nakamu-kazu222 nakamu-kazu222 force-pushed the feature/change-searchables-not-react branch 2 times, most recently from 444ed93 to 7046d46 Compare February 15, 2025 09:42
投稿の種類のラベル、タイトル、summary、ユーザーのアイコン、ユーザー名、更新時間
投稿の幅をそろえた
エラー内容:ActionController::UrlGenerationError in Users#show
No route matches {:action=>"show", :controller=>"talks", :id=>nil}, possible unmatched constraints: [:id]
searchable_urlでコメントのもとのモデルのpathを参照できるようにした
テストの意図、結果を変えずに新しい検索(Searcher.search)の実装に変更した
current_userを渡すようにした
「マルマル シャインカ」のようなスペースを含む検索ワードがそのまま1つの文字列として扱われ、部分一致検索ができるようにした
@nakamu-kazu222 nakamu-kazu222 force-pushed the feature/change-searchables-not-react branch from 0c193c1 to 9938ba8 Compare February 24, 2025 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant