fjordllc / bootcamp

プログラマー向けEラーニングシステム
https://bootcamp.fjord.jp
MIT License
286 stars 71 forks source link

退会者の登録情報変更ページにて、所属企業選択が表示されるように変更 #8140

Closed ayu-0505 closed 3 weeks ago

ayu-0505 commented 1 month ago

Issue

概要

退会済みの場合はユーザー情報変更画面に所属企業の選択欄が表示されないため、変更できない状態であったのを表示されるようにしました。 退会ユーザー本人による情報変更ではなく、管理者としてユーザー情報を変更する際を想定しています。

変更確認方法

  1. bug/display_change_company_form_for_retired_userをローカルに取り込む
    1. git fetch origin pull/8140/head:bug/display_change_company_form_for_retired_user
    2. git switch bug/display_change_company_form_for_retired_user
  2. foreman start -f Procfile.devでローカルサーバーを立ち上げる
  3. 管理者としてログイン(例:komagata
  4. 任意の退会者のページのステータス変更欄にある「管理者として情報変更」をクリック。
  5. 所属企業の変更欄が表示されているか確認。

Screenshot

変更前

スクリーンショット 2024-10-16 14 27 02

変更後

スクリーンショット 2024-10-22 11 12 37
ayu-0505 commented 1 month ago

@su-su-su-su さん、お疲れ様です🍵

もしよろしければこちらのPRのレビューをお手隙の際にお願いできたらと思います。 ご都合が悪い場合は遠慮なくお知らせください🙏

su-su-su-su commented 1 month ago

@ayu-0505

お疲れ様です!承知いたしました。 1週間程でレビューさせていただきます。 よろしくお願いいたします!

su-su-su-su commented 1 month ago

@ayu-0505

お疲れ様です!挙動を確認させていただきました。 kensyuowataの「ユーザー登録情報変更」ページをみた時に、気になった点が1つありました。 今回の変更で、管理者向けの「所属企業」プルダウンの表示が以下のように変わっているようです。

  1. 変更前は、「以下管理者のみ操作ができます」の一番下に「所属企業」プルダウンが表示されていました。 Image from Gyazo
  2. 変更後は、管理者向け部分の直前にも「所属企業」プルダウンが追加され、管理者向け部分にある「所属企業」プルダウンの2箇所に表示されるようになっています。

また、もう一人の退会しているユーザーyameo の「ユーザー登録情報変更」ページを確認してみたのですが、kensyuowataの時と挙動が違っていましたので、ご確認お願いしたいと思っております🙏 よろしくお願いいたします🙇‍♂️

ayu-0505 commented 3 weeks ago

@su-su-su-su さん、確認いただきありがとうございました。 こちらのPRですが、私がissue内容を誤認していたかもしれません。 修正に時間がかかる可能性ありますので、確認と修正を行い次第、改めて再度ご連絡いたします。 お時間いただいて申し訳ありません🙇🏻‍♀️

su-su-su-su commented 3 weeks ago

@ayu-0505 なるほどですね。 ではご連絡お待ちしております🙏

ayu-0505 commented 3 weeks ago

@su-su-su-su さん、お疲れ様です🍵

こちらのPRについてですが、「現在の本番環境においてもIssueにあった内容は確認されず、問題ない」ことがわかりました。 もとのIssueがclosedになったため、こちらのPRもclosedさせていただきます。 色々とお手数をおかけして申し訳ありません🙇🏻‍♀️ レビュー対応ほんとうにありがとうございました。

su-su-su-su commented 2 weeks ago

@ayu-0505 なるほどですね。ご説明ありがとうございます。 いえいえ、全然大丈夫です🙆