bootcamp
bootcamp copied to clipboard
fjordllc
searchable.vue, searchables.vueをVueMounterに対応させる
下記を参考にしてVueMounterを使ったマウント方法に変える。
- Vue Componentのマウント方法 · fjordllc/bootcamp Wiki
- announcementsをVueMounterに対応 by komagata · Pull Request #5113 · fjordllc/bootcamp
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
メモ：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
https://github.com/fjordllc/bootcamp/blob/45988d7098c124e43e34d994d15cbe442fc32440/app/helpers/search_helper.rb#L14-L20
本Issueで扱うSearchables, Searchable関連以外のファイルの以下箇所でこのsearchable_urlが使用されているが、
質問の回答はWatch対象ではないため、今回の変更によって影響を受けない
https://github.com/fjordllc/bootcamp/blob/ec5114627f24fd6fa574923b40a3c94e3acd13b3/app/views/api/watches/_watch.json.jbuilder#L8
@komagata @machida
【ご依頼】本番環境での動作確認について
お疲れ様です！以下のPRが本日マージ&リリースされました。 searchables.vueとsearchable.vueをReactコンポーネントに書き換えた #7218
管理者のみが確認できる画面のため、本番環境でのご確認をお願いできますでしょうか🙏 お手数をおかけいたしますがよろしくお願いいたします🙇♀️
確認方法
- ログインした状態で以下の条件にて検索を行う
- 検索対象：ユーザー
- 検索ワード：nicole2525（またはその他任意のユーザー）
- 1の検索結果として表示されたユーザーの相談部屋のリンクが表示され、クリックするとユーザーの相談部屋に飛ぶことを確認する
@komagata 確認OKとのこと安心いたしました🙏 早々にご確認をいただきありがとうございました🙇♀️！
本番での動作確認が取れたのでcloseいたします！
@komagata @machida
休会中に失礼いたします🙇♀️
こちらのIssueを対応した際に、
Reactに置き換えた元のvueファイル（searchables.vueとsearchable.vue）を削除し損ねていることに気がつきました。
大変申し訳ありません。
期間いっぱい休会するとなると5/5頃の復帰になってしまうのですが、
差し支えなければ復帰後に上記2ファイル（searchables.vueとsearchable.vue）の削除を行うPRを出そうかと考えております。
早めに対応が必要というご状況でしたら、恐れ入りますが他の方に対応いただくことになってしまうかと存じます。
お忙しいところお手数をおかけしてしまい申し訳ありません。 こちらとしてはご返信は急ぎませんので、ご都合のよろしいタイミングでご確認をいただけますと幸いです。 どうぞよろしくお願いいたします🙇♀️
@komagata ご確認いただき、またご対応をいただけるとのことありがとうございます！ お忙しいところお手数をおかけしてしまい申し訳ありません。 恐れ入りますが、どうぞよろしくお願いいたします🙇♀️