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

管理者側のユーザー一覧にある項目を変更 #7413

Closed
wants to merge 4 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Feb 20, 2024

Issue

概要

管理者側のユーザー一覧にある項目を変更しました。
「職業」と「相談部屋」のカラムが追加され、「卒業」と「外部サービス」のカラムは削除されました。
「相談部屋」カラムのリンクをクリックすると、当該ユーザーの相談部屋に遷移するようになっています。

変更確認方法

  1. fearure/modify-user-list-item-in-admin-pageをローカルに取り込む
  2. admin/users/ にアクセスする
  3. 「職業」および「相談部屋」のカラムが表示されていることを確認する
  4. 「卒業」および「外部サービス」のカラムが表示されていないことを確認する
  5. 「相談部屋」カラムのリンクをクリックすると、当該ユーザーの相談部屋に遷移することを確認する

Screenshot

変更前

image

変更後

image

@ghost ghost assigned ghost and machida Feb 20, 2024
@ghost
Copy link
Author

ghost commented Feb 20, 2024

@machida
お疲れ様です。
#7410 の作業が一旦完了したので、デザインをお願いします。🙇‍♂️

@machida
Copy link
Member

machida commented Feb 29, 2024

@taco-nantai デザイン進めますー

@machida
Copy link
Member

machida commented Feb 29, 2024

@taco-nantai 少しだけデザインをいじりました。レビューに進めてください🙏

@ghost
Copy link
Author

ghost commented Mar 2, 2024

@machida
デザインの対応ありがとうございます🙇‍♂️

@ghost
Copy link
Author

ghost commented Mar 2, 2024

@a-kuroki-gs
お疲れ様です。
こちらのPRのレビューをお願いしてもよろしいでしょうか🙏
お手すきの際で構いませんので、よろしくお願いいたします🙇‍♂️

@ghost ghost requested a review from a-kuroki-gs March 2, 2024 01:37
@ghost ghost marked this pull request as ready for review March 2, 2024 01:37
@a-kuroki-gs
Copy link
Contributor

@taco-nantai
お疲れ様です!
今週レビューの時間が取れそうにないので、他の方にレビューをお願いしてもよろしいでしょうか?
申し訳ありません🙏

@ghost
Copy link
Author

ghost commented Mar 4, 2024

@a-kuroki-gs
かしこまりました!
お手数おかけして申し訳ありません🙇‍♂️

@ghost ghost requested review from naokinaokiboo and removed request for a-kuroki-gs March 4, 2024 02:43
@ghost
Copy link
Author

ghost commented Mar 5, 2024

@naokinaokiboo
お疲れ様です。
こちらのPRのレビューをお願いしてもよろしいでしょうか🙏
お手すきの際で構いませんので、よろしくお願いいたします🙇‍♂️

@machida machida removed their assignment Mar 5, 2024
@naokinaokiboo
Copy link
Contributor

@taco-nantai
お疲れ様です。
レビューの件、承知致しました。👍
遅くても明後日には着手する予定ですので、少々お待ち下さい🙏

@naokinaokiboo
Copy link
Contributor

naokinaokiboo commented Mar 8, 2024

@taco-nantai
お疲れ様です。確認致しました。
1点だけ、FBCで定めているブランチ名のルールの部分にタイポがありました。
誤:fearure
正:feature

動作とコードの変更内容を確認し、こちらには問題ないと思いますので、私からはapproveさせて頂きます。👍

@ghost
Copy link
Author

ghost commented Mar 8, 2024

@naokinaokiboo
お疲れ様です。
お忙しい中レビューありがとうございました🙇‍♂️

@komagata
Copy link
Member

komagata commented Mar 9, 2024

@taco-nantai こちらのPRはブランチ名が間違っていたために別のPRを作り直したということでしょうか?
もしそうであればこちらのブランチは不要なので削除をお願いします。
(後から見た時に上記のコメントの流れだけだと何が起こったのか確実にはわからないのでメモを残してもらえるとありがたいです。)

@ghost
Copy link
Author

ghost commented Mar 9, 2024

@komagata
了解しました!

@ghost
Copy link
Author

ghost commented Mar 9, 2024

こちらのPRはブランチ名が間違っていたために別のPRを作り直しました。
作り直したPRは以下になります。

@ghost ghost deleted the fearure/modify-user-list-item-in-admin-page branch March 9, 2024 14:10
This pull request was closed.
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