hirano-vm4

@machida 早速ありがとうございます！ では、そのように進めさせていただきます💪

### メモ📝 ブログ削除後に、メール通知のリンクをユーザーが踏んだ場合の処理 ブログが削除された場合、ユーザーが削除後にメール通知のブログリンクを踏んだ際、ブログ一覧へリダイレクトされ、アラートでユーザーにブログがないことを知らせる処理をしていた。 しかし、[こちらのIssue](https://github.com/fjordllc/bootcamp/issues/7526#issuecomment-2021792609)で「通知からリンクへ飛ぶ際に作品が削除されていた場合は404エラーを返す」という仕様にする確認がなされているため、以下のコミットで変更。 [75b7805](https://github.com/fjordllc/bootcamp/pull/7615/commits/75b78053af8b1279f143efb55ab352e2a7fc6fb7)

@a-terumoto-gs お疲れ様です🙏 お忙しい中、申し訳ありませんがレビューをお願いすることは可能でしょうか🙆？ 急ぎではありませんので、無理ないタイミングで全く問題ありません！(1~2週間後とかでもOKです) ご検討のほど、よろしくお願いいたします🙇‍♂️

@a-terumoto-gs お忙しい中すいません🙏こちらいかがでしょうか？

@a-terumoto-gs お忙しん中、確認いただきありがとうございました🙏引き続きよろしくお願いします！ @komagata こちらチームメンバーによるレビューが終わりました！お忙しいところ、恐れ入りますがレビューをお願いいたします🙏

@machida お疲れ様です！こちらのIssueの実装が完了したので、デザインをお願いします🙏 ## デザインの対象 今回は、メンターの公開用プロフィールをトップページに非表示にできるようにする新機能の実装です。メンターの`current_user/edit`内のチェックボックスで公開・非公開を切り替えます。 - `label`のタイトル - チェックボックス - チェックボックス下の説明文 上記3点がデザインの対象になります！ ## 現在の状態 対象ファイル：`app/views/users/form/_hide_mentor_profile.html.slim` ``` .form-item = f.label :hide_mentor_profile, class: 'a-form-label' label.a-on-off-checkbox.is-md = f.check_box :hide_mentor_profile span .a-form-help p |...

@komagata @machida 先ほどは開発ミーティングありがとうございました！ プロフィールの公開・非公開の切り替え機能に加えて、 「公開プロフィール欄の特定の値が埋まっていない場合」は保存できないようにバリデーションをモデルにかけて、埋めるようユーザーに促す機能を追加するというお話があったと思います🙏 終わった後にslimのコードを確認したところ、メンターさんを表示する条件は以下のようになっています。 `app/views/welcome/_mentors.html.slim` ``` section.welcome-child-section h3.welcome-child-section__title | メンター ・ 顧問 .welcome-members .row.is-gutter-width-32 - mentors.each do |mentor| - if mentor.profile_image.present? && mentor.profile_name.present? && mentor.profile_job.present? && mentor.profile_text.present? &&...

@machida 遅くなのに、早速の回答ありがとうございます🙏 >気付いてくれてありがとうございます！あちゃさんはメンターではないけど、ブログ記事を公開させる必要があったのでメンター権限を付与する必要があり、メンターのロールを持っているが非表示にしていました。 なるほど！あちゃさんは特殊なポジションで活躍されていた事情があったのですね👀 >今回の機能で公開をオフにすればよくなるので、あちゃさんの分岐は削除し、バリデーションにも含めない、でお願いします🙏 承知しました！ それでは実装が本番環境にマージされる際には、非公開を`false`（＝公開する）がデフォルトで設定されるようになるので、あちゃさん含め、メンターさん各位には周知もしたほうがよいかもしれませんね🧐 プライベートなことに絡む部分だと思いますので（会社や特定の人にバレたくないなど）、私が動ける範囲で周知などありましたら、教えていただければお知らせ等したいとおもいますので、何かあればおっしゃってください！

@machida @komagata ## 追記 先程の町田さんの回答で > 今回の機能で公開をオフにすればよくなるので、あちゃさんの分岐は削除し、バリデーションにも含めない、でお願いします🙏 「あちゃさんの分岐は削除」とありましたのでほかは残すという理解をしました！失礼しました🙇‍♂️ 以下は念の為残しますので、認識ズレていたらご助言ください🙏 ## 元々の質問 何度も申し訳ありません！ 先ほどバリデーションを追加して「view側のif文でメンターさんを表示するかどうか条件指定している部分を簡素化する」と私が話してしまいましたが、 ちなみに現在、他に非表示を希望しているメンターさんはいますでしょうか？（ここに書けない場合は、他でご連絡いただければと思います） https://bootcamp.fjord.jp/welcome を見る限りは問題なさそうに見えますが念の為確認させてください🙏 理由としては、この機能は公開がデフォルトになるので、slim側のif文の分岐をなくしてしまうと、これから必須になる項目が現在完全に入力されていない場合でも、トップページに表示されてしまうかもしれません🧐（おそらく次に編集を保存するまでは現在のまま設定でいけてしまう） `mentor.login_name != 'achamixx'`の条件に関してはユーザー設定から非公開を選択することができるため、slimのif文の分岐から削除できると思いますが、他は残した方がよさそうでしょうか？（それでも一部が埋まっていれば非公開を設定するまで表示される可能性があります） あえて表示したくなくて完全に項目を埋めていないが、非公開を設定するまで表示されてしまうケースは0ではないと思っての確認になります（おそらくそのような人はいないと思われますが）、念のためプライベートな部分に関わるので確認させてください🙏 この辺を考えてたバリデーションは設定するが、slim側のif文も残しておくのも選択肢としてベターかなと考えております！（そもそもslim側のif文を簡素化するは私がイメージしていた部分なので、お話と私の捉えがずれていたらすいません） `app/views/welcome/_mentors.html.slim` ``` section.welcome-child-section h3.welcome-child-section__title | メンター ・ 顧問...

@machida お忙しいところ返信ありがとうございます🙏 > 誰がそのケースなのかは把握しているので view の if の分岐は削除してしまって大丈夫です🙆‍♂️リリース直後は表示したくないものが表示されたりなどあるかもですが、すぐに修正します💪 承知しました！では、slim側のif文はすべて削除しておきます🙆 >表示順を created_at 順にしたいと思います。もし、そのようになってなかったら修正をしておいていただきたいです🙏 こちらについては、ログイン状態・非ログイン状態どちらも`created_at`の昇順（古いものから新しいものへ）に並べ替えは完了しています😄 `app/models/user.rb` ```ruby scope :mentors_sorted_by_created_at, lambda { with_attached_profile_image .mentor .includes(authored_books: { cover_attachment: :blob }) .order(:created_at) } scope :visible_sorted_mentors,...