bootcamp
bootcamp copied to clipboard
fjordllc
メンターのプロフィールを追加する
Issue
- #4855
概要
【現在】
トップページの「メンター・顧問」のプロフィールカードにDBが使用されていない。
【変更点】
- 登録情報変更画面 (ユーザー登録情報変更画面) にメンターのみプロフィール用の入力スペースを表示する。
- 登録情報変更画面 (ユーザー登録情報変更画面) に入力したデータをusersテーブルに格納する。
- usersテーブルのmentor: trueのアカウントをeachで回してトップページの「メンター・顧問」にプロフィールカードを表示する。
※ 全て (プロフィール画像、プロフィール名、職業、プロフィール文) 入力したらプロフィールカードが表示されます。 ※ snsアカウントはnilの場合、表示されません。
今回usersテーブルに追加したカラム
- profile_name
- profile_job
- profile_text
※ profile_imageはActive Storageを使用しています。 ※ profile_textはマークダウンに対応しています。 ※ snsアカウントは既存のusersテーブルのカラムを使用しています。
変更確認方法
【登録情報変更画面からプロフィールを入力する場合】
- ブランチ
feature/add-mentor-profileをローカルに取り込む -
bin/rails sでローカル環境を立ち上げる - メンターアカウントでログインし、http://localhost:3000/current_user/edit にアクセスする。
- プロフィール画像、プロフィール名、職業、プロフィール文を入力して、更新する。
- ログアウトして、http://localhost:3000/welcome にアクセスする。
【ユーザー登録情報変更画面からプロフィールを入力する場合】
- ブランチ
feature/add-mentor-profileをローカルに取り込む -
bin/rails sでローカル環境を立ち上げる - 管理者アカウントでログインし、http://localhost:3000/admin/users?target=mentor にアクセスする。
- 「操作」をクリックする。
- プロフィール画像、プロフィール名、職業、プロフィール文を入力して、更新する。
- ログアウトして、http://localhost:3000/welcome にアクセスする。
変更前
変更後
@tomonariha お疲れ様です。 お手隙の際に、こちらのレビューをお願いしたいです🙏 ご不明な点やご都合が合わないようでしたら、お気軽にご連絡ください！
@daiki0381 さん、修正ありがとうございます！ コメントしましたので、ご確認ください🙏
あと、このPRのコミットがおかしくなっているようです。(他の人のコミットが大量に入り込んでいます)
git pull --rebase origin mainではなく、別のコマンドで変更を取り込んだりしましたか？
ご自身で修正できそうなら修正してみてください🙏
私もあまりgit操作に自信がないので、無理そうならメンターの方に相談されるのが確実かと思います🙇♂️
@daiki0381 さん、修正ありがとうございます🙏 テスト環境の初期データを用意したのいいですね！こちらの動作確認もしてみて、問題なさそうでした🙆♂️ 1点、画像のアタッチメントで気になる事があるのでコメントしておきました。 ご確認お願いします🙇♂️
@daiki0381 返信遅れてすいません。
こちら、続きのIssueを作ります。新たなIssueのPRはこちらのブランチにマージするようにします。
デザインを反映させるのでもう少々お待ちください🙇♂️
📝
https://github.com/fjordllc/bootcamp/issues/5287
こちらのIssueの詳細を書き、PRの送り先をこのブランチに設定する。
📝 https://gist.github.com/machida/61003a017e610348382489916d2d5339 リリース前に消したファイルを一旦戻し、旧メンター紹介を表示させ、新しいメンター紹介はコメントアウトにする。 メンター情報を入力し、新しいメンター紹介が表示できるようになったら、コメントアウトを外し、古いファイルは削除する。
@machida app/assets/stylesheets/application/blocks/form/_reference-books-form.sassでコンフリクトが発生しているので、ご確認お願いしたいです🙏
@daiki0381 (CC: @machida)
このままリリースすると、本番環境ではデータが入るまではメンターのプロフィールが空か意図しないないようになってしまうように思います。
表示部分についてはコメントアウトしてリリースし、データが入ったことを確かめたからそこのコメントを外すのはいかがでしょうか？ > @machida
@komagata 了解です🙆♂️ 一旦、表示部分を元に戻しました。本番環境にデータが入ったことを確認してから、別issueで修正しようと思います。また、元に戻している影響でテストが落ちています。
@daiki0381 @komagata すいません、遅くなりました。 こちら、新しい表示はコメントアウトして、管理者でログインしたときだけ見れるようにします。 テストのために、env が test のときは新しい表示を見るように一時的な分岐を作っておきますー
@daiki0381 @komagata 本番環境では古い表示、テスト・開発環境では新しい表示をするようにしました。 また、本番環境でも管理者でログインをした場合だけ、古い表示・新しい表示の両方を表示するようにしました（データ入力作業をしやすくするため）