Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

検索結果がユーザーだった場合にユーザーアイコンが表示されるようにした #5054

Merged
merged 5 commits into from
Jun 29, 2022

Conversation

tomonariha
Copy link
Contributor

@tomonariha tomonariha commented Jun 20, 2022

Issue

概要

検索結果がユーザーの場合にユーザーアイコンが表示されるようにした

変更確認方法

  1. ブランチfeature/display-user-icon-when-search-result-is-userをローカルに取り込む
  2. bin/rails sでローカル環境を立ち上げる
  3. 任意のユーザーでログインする
  4. ヘッダーにある検索ウィンドウにユーザーと入力し検索する
  5. 検索結果がユーザーの場合に、ユーザーアイコンが表示されているか確認する

変更前

スクリーンショット 2022-06-17 13 50 21

変更後

スクリーンショット 2022-06-20 23 36 38

@tomonariha tomonariha self-assigned this Jun 21, 2022
@tomonariha
Copy link
Contributor Author

@machida さん、お疲れ様です。
ユーザーアイコンが表示されるようになりましたので、デザインの方よろしくお願いします🙏

@machida
Copy link
Member

machida commented Jun 23, 2022

@tomonariha
デザイン入れました!
リファクタをしたので変更箇所が多く、別PRにしました。
#5068
このPRのレビューが取ったら、上記のPRをこのPRにマージしようと思いますー
なので、レビューを進めてください。

@tomonariha
Copy link
Contributor Author

@machida さん、デザインありがとうございます!
別PRの件、了解しました。レビューの方に進みます。

@tomonariha
Copy link
Contributor Author

@ai-24 さん、お疲れ様です。
お時間あるときにレビューよろしくお願いします🙏

@tomonariha tomonariha requested a review from ai-24 June 23, 2022 07:30
@machida machida marked this pull request as ready for review June 23, 2022 08:03
Copy link
Contributor

@ai-24 ai-24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tomonariha さん
お疲れ様です。

確認させていただきました😊
ユーザーアイコンの周りの色の表示は関数を使ってクラス分けしているのですね!
勉強になりました!レビュー依頼ありがとうございました🙇‍♀️

私からはApproveさせていただきます!

@tomonariha
Copy link
Contributor Author

@ai-24 さん、レビューありがとうございます🙏
私もアプリの他の部分を参考にしたのですが、ユーザーのロールを関数の中でデコレータを使って決めてるんだ〜😳と思いながら実装してました!

@tomonariha tomonariha force-pushed the feature/display-user-icon-when-search-result-is-user branch from dd29ada to 58ee594 Compare June 26, 2022 13:41
@tomonariha
Copy link
Contributor Author

@komagata さん、お手隙の際にレビューよろしくお願いします🙏

@machida
Copy link
Member

machida commented Jun 27, 2022

@tomonariha #5068 こちらが先にマージされたので、mainにマージされるとデザインが当たるようになりますー
main にマージ後、何か見た目の不具合が出ましたら修正します。

@machida machida removed their assignment Jun 27, 2022
@tomonariha tomonariha requested a review from komagata June 27, 2022 13:38
@tomonariha
Copy link
Contributor Author

@machida さん、了解しました。
マージ後、もしデザイン関係の不具合が出たらご連絡します。

Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

確認させて頂きました。OKです〜🙆‍♂️

@komagata komagata merged commit 57e8d37 into main Jun 29, 2022
@komagata komagata deleted the feature/display-user-icon-when-search-result-is-user branch June 29, 2022 00:35
@github-actions github-actions bot mentioned this pull request Jun 29, 2022
22 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants