searchable.vue, searchables.vueをreactに対応させる

[komagata]

下記を参考にしてreactに変える。

• react導入 + 管理画面のコース一覧をreact化 by komagata · Pull Request #5515 · fjordllc/bootcamp

[github-actions]

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

[github-actions]

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

[github-actions]

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

[github-actions]

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

[github-actions]

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

[github-actions]

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

[github-actions]

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

[github-actions]

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

[2525nicole]

メモ：2024/1/17のチーム開発MTGでご相談した内容

ご相談内容

右上の検索窓を使用した検索結果でQ＆Aの回答がヒットした際に、
リンクが「現状：紐づくQ＆AのURLになっている」のを「回答自体のURL」に変更しても問題ないか。

↓右上の検索窓

経緯

React化するにあたってSearchablesコンポーネントから個々の検索結果であるSearchableコンポーネントにkeyを渡す必要がある
=>それぞれの検索結果のidをkeyにすると、異なるモデル同士の同じidが検索結果としてヒットした時にkeyが一意でなくなってしまう可能性がある
=> それぞれのURLをkeyとして渡そうと考えた

但し、現在の仕様（回答のリンクは紐づくQ＆Aのリンクになっている）だと、
質問と回答両方がヒットした時にいずれも質問のURLがkeyになってしまうため一意ではなくなってしまう
=>質問の回答自体のURLを`keyとして渡せれば解決すると考えた

頂戴したご回答

回答自体のURLに変更して問題ない

メモ

回答がヒットした時に質問のURLに変換する処理は以下で行われている

app/helpers/search_helper.rb内のdef searchable_url

bootcamp/app/helpers/search_helper.rb

Lines 14 to 20 in 45988d7

	def searchable_url(searchable)
	if searchable.instance_of?(Comment)
	document = searchable.commentable_type.constantize.find(searchable.commentable_id)
	"#{polymorphic_url(document)}#comment_#{searchable.id}"
	elsif searchable.instance_of?(Answer) || searchable.instance_of?(CorrectAnswer)
	document = searchable.question
	polymorphic_url(document).to_s

本Issueで扱うSearchables, Searchable関連以外のファイルの以下箇所でこのsearchable_urlが使用されているが、
質問の回答はWatch対象ではないため、今回の変更によって影響を受けない

bootcamp/app/views/api/watches/_watch.json.jbuilder

Line 8 in ec51146

json.url searchable_url(watch.watchable)

[2525nicole]

@komagata @machida

【ご依頼】本番環境での動作確認について

お疲れ様です！以下のPRが本日マージ&リリースされました。
searchables.vueとsearchable.vueをReactコンポーネントに書き換えた #7218

管理者のみが確認できる画面のため、本番環境でのご確認をお願いできますでしょうか🙏
お手数をおかけいたしますがよろしくお願いいたします🙇‍♀️

確認方法

1. ログインした状態で以下の条件にて検索を行う

• 検索対象：ユーザー
• 検索ワード：nicole2525（またはその他任意のユーザー）

2. 1の検索結果として表示されたユーザーの相談部屋のリンクが表示され、クリックするとユーザーの相談部屋に飛ぶことを確認する

[komagata]

@2525nicole 本番環境で動作確認してOKでした〜。

[2525nicole]

@komagata
確認OKとのこと安心いたしました🙏
早々にご確認をいただきありがとうございました🙇‍♀️！

本番での動作確認が取れたのでcloseいたします！

[2525nicole]

@komagata @machida
休会中に失礼いたします🙇‍♀️
こちらのIssueを対応した際に、
Reactに置き換えた元のvueファイル（searchables.vueとsearchable.vue）を削除し損ねていることに気がつきました。
大変申し訳ありません。
期間いっぱい休会するとなると5/5頃の復帰になってしまうのですが、
差し支えなければ復帰後に上記2ファイル（searchables.vueとsearchable.vue）の削除を行うPRを出そうかと考えております。
早めに対応が必要というご状況でしたら、恐れ入りますが他の方に対応いただくことになってしまうかと存じます。

お忙しいところお手数をおかけしてしまい申し訳ありません。
こちらとしてはご返信は急ぎませんので、ご都合のよろしいタイミングでご確認をいただけますと幸いです。
どうぞよろしくお願いいたします🙇‍♀️

[komagata]

@2525nicole ご連絡ありがとうございます。
こちらで対応しますので大丈夫です〜

#7430

[2525nicole]

@komagata
ご確認いただき、またご対応をいただけるとのことありがとうございます！
お忙しいところお手数をおかけしてしまい申し訳ありません。
恐れ入りますが、どうぞよろしくお願いいたします🙇‍♀️

[komagata]

@2525nicole こちらで 対応しました〜
#7430

[2525nicole]

@komagata
当該ファイルを削除いただいたこと確認できました🙇‍♀️
早速のご対応ありがとうございました😭🙏！