ユーザー個別ページ、一覧ページのカウント数一覧にポートフォリオを追加

[unikounio]

Issue

• https://github.com/fjordllc/bootcamp/issues/7517

概要

ユーザー個別ページ、一覧ページのカウント数一覧にポートフォリオを追加しました。 また、ユーザー個別ページのカウント数一覧において、カウント数が0の場合はリンクしないよう変更しました。

変更確認方法

1. feature/add-portfolio-count-to-userをローカルに取り込む
2. bin/setupを実行
3. foreman start -f Procfile.devでサーバーを立ち上げる
4. 任意のアカウントでログインし、/usersにアクセス
5. 次の2点を確認する
   • 各ユーザーのカウント数一覧に「ポートフォリオ」項目が追加されている
   • カウントの数字部分のリンクが次の状態になっている
     • カウントが0ではない場合 ユーザー個別ページの対応するタブにリンクにしている。
     • カウントが0の場合 リンクなし。
6. /usersからユーザーhatsunoをクリック
7. 次の2点を確認する
   • カウント数一覧に「ポートフォリオ」項目が追加されている
   • カウントの数字部分のリンクが次の状態になっている
     • カウントが0ではない場合 対応するタブにリンクにしている。
     • カウントが0の場合 リンクなし。

Screenshot

変更前

/users

/users/show

変更後

/users

/users/show

[unikounio]

@kurumadaisuke お疲れ様です！ こちらのPRのレビューをお願いさせていただきたくてご連絡させていただきました🙏 ご対応可能なようでしたら、お手すきの際にでもご確認いただけますと幸いです。 よろしくお願いいたします。

[kurumadaisuke]

@unikounio すいませんmm ちょっと仕事がバタバタしていてまとまった時間を取ることが難しく、、、 他の方でもよろしかったでしょうか🙏

[unikounio]

@kurumadaisuke さん お忙しいところすいません！ ご連絡ありがとうございます～🙏✨ 別の方にお願いさせていただきますね。

@a-kuroki-gs さん お疲れ様です！ こちらのPRのレビューをお願いさせていただきたくてご連絡させていただきました🙏 ご対応可能なようでしたら、お手すきの際にでもご確認いただけますと幸いです。 よろしくお願いいたします。

[a-kuroki-gs]

@unikounio お疲れ様です！ レビュー依頼ありがとうございます。

ただ、大変申し訳ないのですが、現在レビューの時間を取ることが厳しいため、 他の方への依頼をお願いしてもよろしいでしょうか？😭

[unikounio]

@a-kuroki-gs さん ご連絡ありがとうございます！🙏✨ ご多用のところ依頼してしまってすいませんでした🙏 別の方にお願いさせていただきます～

@hirano-vm4 さん お疲れ様です！ こちらのPRのレビューをお願いさせていただきたくてご連絡させていただきました🙏 けだまさんからのご依頼をお断りしたのにこちらからご依頼する形になってしまい申し訳ないのですが、ご対応可能なようでしたら、お手すきの際にでもご確認いただけますと幸いです😅 よろしくお願いいたしますｍｍ

[hirano-vm4]

@unikounio

もちろん大丈夫です🙆お時間数日いただければと思います！よろしくお願いします🙏

[hirano-vm4]

@unikounio

お疲れさまです！遅くなりました！早速コードを確認しました😄実装のコードは自体はわかりやすく、良いコードだと感じましたが、1点だけ気になった点を共有したいと思います🙏

ユーザー一覧での表示ズレ

http://localhost:3000/users/

• 問題ない表示

• 要素の高さがずれている表示

私の環境に固有の現象かもしれませんが、ブラウザのサイズを変更するとactivity-nameに設定された「ポートフォリオ」という文字が他の文字に比べて多いため、横幅がギリギリになっています。

その結果、ブラウザのウィンドウサイズによっては文字列が2行表示になり、レイアウトが崩れてしまうようです🧐

app/javascript/components/user-activity-counts.vue

    user-activity-count(
      activity-name='ポートフォリオ',
      :activity-count='user.work_count',
      :activity-url='`${user.url}/portfolio`')

固有の問題の可能性もあるので @unikounio さんの環境でも念の為確認していただき、状況によっては町田さんにこのままいくかどうかの確認を入れても良いかも？と感じましたがいかがでしょうか？🙆

確認チェック

• [x] カウント数一覧に「ポートフォリオ」項目が追加されている
• [x] カウントの数字部分のリンクが次の状態になっている
• [x] カウントが1以上の場合、対応するタブにリンクにしている。
• [x] カウントが0の場合 リンクなし

[unikounio]

@hirano-vm4 さん ご確認ありがとうございます！ ご指摘の点については実は私も気になったので確認していました。 本番環境での動きを見たところ、activity-nameの表示は元からこのようになっているようなので、この実装には意図がある、もしくは整理するとしても別Issueになるだろうと考え、変更せずにいた次第です。

【本番環境での表示】

とはいえ２人とも気になる点ということで、念のため町田さんに確認させていただこうと思います😄

[unikounio]

~~@machida さん お疲れ様です！ ユーザー一覧画面での表示について、実装方針の確認をさせてください🙏 現在の実装ではユーザー一覧画面（/users）を表示する際に、ブラウザのサイズ次第で要素名が2列になって表示されるようになっているかと思います。~~

~~【本番環境での表示】~~

~~これについて本PRでは特段の調整を加えない方針で進めているのですが、この点いかがでしょうか？ お手すきの際にご回答いただけますと幸いです。~~

3/27 追記 本件は本日のチーム開発MTGにて町田さんご確認済み

[hirano-vm4]

@unikounio お疲れ様です！もとからそのような状況だったのですね👀コード的には私は問題ないと思いましたので、確認できたらApproveさせてもらいます🙏

[unikounio]

@hirano-vm4 さん お疲れさまです！ 要素名の表示ズレの件について、本日のチーム開発MTGで町田さんに確認していただき、別Issueで進めることとなりました。

• ユーザー一覧ページのカウント数一覧における要素名の表示を固定したい · Issue #7606 · fjordllc/bootcamp

お手すきの際にご確認をお願いいたします🙏

[unikounio]

@hirano-vm4 さん レビューありがとうございました！✨ 引き続きよろしくお願いいたします～😄

[unikounio]

@komagata さん お疲れ様です！ チームメンバーからApproveをいただきましたので、お手すきの際にレビューをお願いいたします🙏