管理者は自分以外のユーザーのGitHubアカウント情報を削除できるようにしたい。

[machida]

管理者権限で自分以外のユーザーの編集画面にいき、このリンクをクリックしても、

自分のGitHubアカウントとの紐づけが解除されてしまう。
このユーザーのGitHubアカウントとの紐づけは解除できない。

このリンクをクリックして、自分以外のユーザーのGitHubアカウント情報を削除できるようにしたい。

[Shrimprin]

@komagata @machida
お疲れ様です。本イシューの仕様について2点確認させていただけますでしょうか。

今回のイシューで対応する範囲はGitHub, Discord両方ですか？

イシューのタイトルはGitHubとなっていますが、対応範囲はGitHub, Discordの両方という認識で合っていますでしょうか？

GitHub, Discordの登録削除後の遷移先のページは、削除されたユーザーのプロフィールページとしてよいでしょうか？

現状、登録削除後の遷移先はroot_path(=ダッシュボード)となっています。
他のユーザーの登録を削除した際は、そのユーザーのプロフィールページに遷移させた方が良いのではないかと考えています。
また、それに合わせてログインユーザー自身の登録削除した場合も、ログインユーザーのプロフィールページに遷移させた方が一貫性があると考えています。

下記のどちらが良いかご助言いただけますでしょうか。

• 登録削除されたユーザーに関わらずダッシュボードとする
• 削除されたユーザーのプロフィールページとする

p.s.
誤操作でイシューをクローズしてしまったため、再度オープンし直しました。

以上、よろしくお願いいたします。

[machida]

@Shrimprin 質問ありがとうございます！

今回はGitHubだけにして、Discordは別Issueにしようと思います。

遷移先は確かに、そのユーザーのプロフィールページがいいですね。そのように対応お願いします🙏

[Shrimprin]

@machida
ご回答いただきありがとうございます 🙏
承知致しました！

[Shrimprin]

@machida
度々申し訳ございません、もう1点質問です。

初期データ(seed)にあるusersはすべてgithub_idカラムがnilとなっているため、動作の確認ができない状況です。
手元での動作確認はrails cで適当なuserを作成すればできるのですが、プルリクエストでレビュアーに確認依頼をする際はどのように確認用のデータを用意すべきでしょうか？

下記のどちらが良いかご助言いただけますでしょうか。
私としてはseedに追記して初期データを用意する方が親切と考えています。

• rails cで作成してもらう
• seedファイルにgithub_idを入れたレコードを書いておき、初期データを用意する

お手数ですがご回答よろしくお願いいたします 🙏

[machida]

@Shrimprin
seedでお願いしたいですー。
あと、testも欲しいですね。

[Shrimprin]

@machida
ご返信ありがとうございます！
seedでのデータ作成と、testの作成承知いたしました。

[Shrimprin]

@komagata @machida
こちらの動作確認に管理者権限が必要なため、本番環境での確認をお願いできますでしょうか 🙏

[machida]

@Shrimprin 本番で確認しました🙆対応ありがとうございます！！