くるま

@nishitatsu-dev お疲れ様です。 こちらレビューお願いしてもよろしかったでしょうか？🙏 特に期限は設けていませんのでお手隙の際にご確認いただければと思います🙏

@komagata 本issueの対応方法について質問させてください ## Searchableモジュールの変更方針について FBCアプリの右上に存在する検索処理ではSearchableモジュールを利用しており、約10個のモデルファイルにincludeされています。 その中で`app/models/user.rb`が`scope :search_by_keywords_scope, -> { unretired }`を利用して検索結果を制限しています。 変更方法案① Searchableモジュール自体は変更せず、代わりに利用者側の変更を行います。 具体的には、`app/models/user.rb`内の`scope :search_by_keywords_scope, -> { unretired }`の行を削除します。 変更方法② Searchableモジュール自体を変更し、`scope :search_by_keywords_scope, -> { all }`を削除します。この変更に伴い、利用者側の`app/models/user.rb`でも同じく`scope :search_by_keywords_scope, -> { unretired...

@komagata > まだ判断するための材料がたりないとおもうので、それぞれの影響範囲（実際のどのページにもちいられるか。そのページが変わっても問題ないか）を調べてみてください。 > そのうえで @kurumadaisuke さんがどうすべきだと思うかも考えてみてください。 こちら調査した後に回答いたします。 > メンターの検索というのはどういう意味でしょうか？ 一般ユーザー（受講生）からの「ユーザー」タブを選択した時の表示内容と メンターから「ユーザー」タブを選択するのでは、表示が異なっており（メンターの方が多く表示されている）メンターが「ユーザー」タブを選択した状態をメンターの検索という風に言っていました🙏

@komagata > 「言っていました」というのはどういう意味でしょうか？ すいません。 言葉足らずでした🙏 一般ユーザー（受講生）からの「ユーザー」タブを選択した時の表示内容と メンターから「ユーザー」タブを選択するのでは、表示が異なっており（メンターの方が多く表示されている）メンターが「ユーザー」タブを選択した状態を「メンターの検索」という意味で使った言葉になります🙏

@komagata https://github.com/fjordllc/bootcamp/pull/7435#issuecomment-2014380301 こちら調査してみたのですが、Searchableモジュールの`scope :search_by_keywords_scope, -> { all }`の部分を使っているのが`app/models/user.rb`の`scope :search_by_keywords_scope, -> { unretired }`だけだったので使わないコードはなるべく削除したほうが良いっと考え、大元の`scope :search_by_keywords_scope, -> { all }`を削除して影響範囲（テスト）がどうなるか確認をしました。 テストの結果は一つ失敗しましたが、元々退会/休会ユーザーが表示されないことを確認するテストだったのでアプリ自体の仕様が変わったので、テスト自体を修正しました。 なので調査結果では、大元を削除しても影響範囲がほとんどないっと考え不要なコードは削除するべきだと考えました🙏

@komagata お疲れ様です！！ 本issueについてですが、私が休会中に以下issueで別の方に行っていただいたので本件についてはクローズでよろしかったでしょうか？🙏 ご確認お願いします！ https://github.com/fjordllc/bootcamp/pull/7941

@komagata @machida 昨日アサインしていただいた件についてですが、issue自体がクローズされているようです。 私の方で何か対応が必要なことがあれば、ご確認いただけますでしょうか？🙏

@komagata ご対応ありがとうございました！

@komagata こちらのissueを先行して割り振ってもらったのですが、 https://github.com/fjordllc/bootcamp/issues/7605 と内容一緒ですかね？ 私の場合、ある程度内容調べていた段階なのでコード自体は全く書いていませんmm もし、同じ内容の場合仕事が忙しく手をつけられない期間が続きそうなので私でなくても大丈夫です🙇‍♂️ ※ discodeでも共有させていただいたので念の為 https://discord.com/channels/715806612824260640/809595476847493192/1222446062929973329

@machida ご連絡ありがとうございます！ 引き続き本issue対応いたします！