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

研修生は退会ではなく、研修終了手続きを行うように変更 #8356

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

reckyy
Copy link
Contributor

@reckyy reckyy commented Feb 19, 2025

Issue

概要

研修生はこれまで研修を終了する際、現役生と同じく退会手続きを行なっていた。
新しく研修終了のフローを用意し、研修終了手続きを行うように変更。
参考 :

変更確認方法

  1. feature/add-training-completionをローカルに取り込む
  2. rails db:migrate
  3. rails db:reset
  4. foreman start -f Procfile.dev
  5. kensyuでログイン(研修中のユーザーであれば良いが、以下はkensyuで操作を行う想定で記載します)
  6. ユーザーメニューの「研修終了手続き」、もしくは http://localhost:3000/current_user/editの最下部の「研修終了手続き」から、http://localhost:3000/training_completion/newにアクセス
  7. 満足度と、ご意見を回答し「研修を終了する」をクリック
  8. http://localhost:3000/training_completionに遷移し、研修終了手続きが完了した旨のページの表示を確認
  9. http://localhost:3000/letter_opener にアクセスし、kensyuあてに研修終了手続きが完了した旨のメールが来てることを確認
  10. kensyuでログインできないことを確認
  11. komagataでログイン(管理者 or メンターのユーザーであれば良い)
  12. kensyuの研修が終了した通知が来てることを確認し、同通知をクリック
  13. 研修終了情報が表示されてることを確認

Screenshot

最終的なデザインが確定したのちに、追記します。

変更前

変更後

@reckyy reckyy force-pushed the feature/add-training-completion branch from 609b9a2 to 55521a5 Compare February 21, 2025 08:31
@reckyy reckyy force-pushed the feature/add-training-completion branch from 55521a5 to f237f89 Compare February 21, 2025 09:27
@reckyy
Copy link
Contributor Author

reckyy commented Feb 21, 2025

@machida
お疲れ様です!
実装が完了したので、以下のデザインおよび文言の変更をお願いしたいです。
また、あわせてご確認いただきたい質問がいくつかございますので、ご確認いただけますと幸いです。

デザインをお願いしたい箇所

  • 研修終了手続き完了後のページ

デザインおよび文言は退会完了後のページのものを転用しています。
該当ファイル : app/views/training_completion/show.html.slim
スクリーンショット 2025-02-21 23 36 57

  • 研修終了者のプロフィールページ

プロフィールページ内に表示される、研修終了情報の部分です。
該当ファイル : app/views/users/_training_completion_info.html.slim
スクリーンショット_2025-02-21_23_37_46

  • 研修終了者へ送られる通知メールの題名・本文

現在は仮の文章を入れています。以下のファイルに対して修正をお願いいたします。
本文 : app/views/user_mailer/training_complete.text.erbです。
題名 : app/mailers/user_mailer.rbtraining_completeメソッド(15行目)のsubjectの部分

  • 研修終了者が発生した時の、メンターや管理者への通知の文言
スクリーンショット_2025-02-21_23_37_31

現在は仮の文章を入れています。

  def training_completed(params = {})
    notification(
      body: "#{sender.login_name}さんの研修が終了しました。",
    )
  end

質問

  1. 日報の表示画面

現在、メンターもしくは管理者が退会した人の日報を見ると、「このユーザーは退会しています。」の文言が表示されます。
研修終了者についても、同様に何かしらの文言を表示すべきでしょうか?
スクリーンショット 2025-02-22 0 01 43
d2e2-4be4-b936-826da8a09685" />

  1. 選択入力の研修感想

研修終了情報に表示される「選択入力の研修感想」については、以下の画像の選択肢の日本語をそのまま表示する形でいいでしょうか?
(とても良い、良い、どちらでもない、悪い、とても悪い、の五種類)
<img width="300" alt="スクリーンショット 2025-02-21 23 51 56" src="https://github.com/user-attachments/assets/65cfbf9b-

  1. ユーザーアイコンの枠について

退会済みユーザーのアイコンにデザイン変更があるように、研修修了者に対しても別途デザインを適用すべきでしょうか?(メンターや管理者にのみ表示される)
もし必要でしたら、同時にデザインをお願いいたします。クラス名は、is-training_completedになります。

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.

1 participant