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.

Sign up for GitHub

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

Jump to bottom

管理者でログイン時にユーザー一覧の各ユーザーに相談部屋へのリンクを表示 #4084

Merged
merged 3 commits into from
Feb 3, 2022
Merged

管理者でログイン時にユーザー一覧の各ユーザーに相談部屋へのリンクを表示 #4084

merged 3 commits into from
Feb 3, 2022

Conversation

Paru871
Copy link
Contributor

@Paru871 Paru871 commented Jan 29, 2022 •
edited
Loading

Issue

概要

管理者でログインしたときのみ、ユーザー一覧ページのそれぞれのユーザーの枠内に相談部屋へのリンクを表示させる変更作業を行いました。

お知らせ

このブランチの名前について、feature/add-link-to-talk-on-user-list-only-mentorとなっていますが、最後のところはonly-adminとするべきでした。間違いに気づいたのが遅かったためブランチ名を変更できませんでした。何とぞご了承ください。

変更確認方法

  1. ブランチ feature/add-link-to-talk-on-user-list-only-mentorをローカルに取り込む
  2. $ rails s でローカル環境を立ち上げる
  3. テスト用ユーザー(管理者ロールを持っている'komagata''machida'のどちらか)でログインする
  4. 左側のナビゲーションバーから「ユーザー」をクリックしてユーザー一覧ページを開きます。
  5. それぞれのユーザーの枠内に「相談部屋」の文字と、それぞれの相談部屋へのリンクが付加されていることをご確認ください。
  6. 一旦ログアウトして、次は'komagata''machida'以外のテスト用ユーザーでログインし、ユーザー一覧ページのそれぞれのユーザーの枠内に「相談部屋」の文字とリンクがないこともご確認ください。

変更【前】(「相談部屋」へのリンクが表示されていなかった)のスクショ

image

変更【後】(「相談部屋」へのリンクが表示されている)のスクショ

image

@Paru871 Paru871 marked this pull request as draft January 29, 2022 13:34
@Paru871 Paru871 self-assigned this Jan 29, 2022
@Paru871 Paru871 force-pushed the feature/add-link-to-talk-on-user-list-only-mentor branch from c88835e to 66d15d9 Compare January 29, 2022 14:19
@Paru871
Copy link
Contributor Author

Paru871 commented Jan 29, 2022

@machida
お疲れさまです。管理者にログイン時にユーザー一覧ページの各ユーザーの枠内にそれぞれの「相談部屋」へのリンクを置きました。お手すきの際にデザインをつけていただきたく、よろしくお願いします🙏
(架空のcssのクラスusers-item__linkto_talk.is-only-adminを書いてみましたので、適切な表記に変更もお願いします。)

@machida
Copy link
Member

machida commented Jan 31, 2022

@Paru871 ありがとうございます！これでOKですー デザイン入れます💪

@machida
Copy link
Member

machida commented Jan 31, 2022

@Paru871 デザイン入れましたー
僕の環境のせいかもしれないのですが、ローカルでユーザー一覧が表示されなかったので、何か問題が起きてるかもしれないです。なので、画面で確認できていないのですが、デザインは多分ちゃんと入ってると思いますー

@machida machida removed their assignment Jan 31, 2022
@Paru871
Copy link
Contributor Author

Paru871 commented Jan 31, 2022

@machida
今回も早速にかっこいいデザインをつけてくださりありがとうございました🙇🏻‍♀️
私のローカル環境ではユーザー一覧画面が表示されて、デザインが付いた新しい「相談部屋」のボタンが表示されています。
(description欄の変更後のスクショにてご確認ください。)
それでは次のステップのチームメンバーレビューに進みます。ありがとうございます！

@Paru871 Paru871 requested a review from maeda-seina January 31, 2022 13:12
@Paru871
Copy link
Contributor Author

Paru871 commented Jan 31, 2022

@maeda-seina
お疲れさまです！
お手すきの際にこちらのレビューをいただきたく、よろしくお願いします🙏

@Paru871 Paru871 marked this pull request as ready for review January 31, 2022 13:42
maeda-seina
maeda-seina approved these changes Feb 1, 2022
View reviewed changes
Copy link
Contributor

@maeda-seina maeda-seina left a comment

Choose a reason for hiding this comment

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

@Paru871 さん
レビュー依頼ありがとうございます。
動作確認できました！丁寧に記述いただいていて、わかりやすかったです〜!!
LGTMです！👍

@Paru871
Copy link
Contributor Author

Paru871 commented Feb 2, 2022

@maeda-seina
早速にご確認くださりありがとうございました！OKをいただけて安心しました〜😊
今後ともよろしくお願いします🙏🏻

@Paru871
Copy link
Contributor Author

Paru871 commented Feb 2, 2022

@komagata
お疲れさまです。チームメンバーさんのレビューのOKをいただきました。
お手すきの際にレビューをよろしくお願いします🙇‍♀️

komagata
komagata requested changes Feb 2, 2022
View reviewed changes
app/views/api/users/_list_user.json.jbuilder Outdated
@@ -14,6 +14,7 @@ json.student_or_trainee user.student_or_trainee?
json.edit_admin_user_path edit_admin_user_path(user)
json.isFollowing current_user.following?(user)
json.isWatching current_user.watching?(user)
json.talkurl talk_path(user.talk)
Copy link
Member

Choose a reason for hiding this comment

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

camelcaseの名前にした方がいいかもです〜。（名前が判別しづらいので）

@Paru871 Paru871 force-pushed the feature/add-link-to-talk-on-user-list-only-mentor branch from dd5a076 to 8d81438 Compare February 2, 2022 07:43
@Paru871
Copy link
Contributor Author

Paru871 commented Feb 2, 2022

@komagata
レビューありがとうございます！
キャメルケースを忘れてしまい、失礼しました。
間違えた2箇所をキャメルケースtalkUrlに修正しましたので、再度チェックいただきたくよろしくお願いします🙏

@Paru871 Paru871 requested a review from komagata February 2, 2022 08:09
komagata
komagata approved these changes Feb 3, 2022
View reviewed changes
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 b13c596 into main Feb 3, 2022
@komagata komagata deleted the feature/add-link-to-talk-on-user-list-only-mentor branch February 3, 2022 08:28
@github-actions github-actions bot mentioned this pull request Feb 3, 2022
Merged
54 tasks
This was referenced Feb 18, 2022
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maeda-seina maeda-seina maeda-seina approved these changes

@komagata komagata komagata approved these changes

Assignees

@Paru871 Paru871

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Paru871 @machida @komagata @maeda-seina