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

招待URL作成ページが欲しい #7424

Closed
komagata opened this issue Feb 22, 2024 · 12 comments
Closed

招待URL作成ページが欲しい #7424

komagata opened this issue Feb 22, 2024 · 12 comments
Assignees
Labels

Comments

@komagata
Copy link
Member

komagata commented Feb 22, 2024

現状では管理画面の企業一覧ページにアドバイザー登録URLと研修生登録URLがあるが、他にもたくさん種類があるので、それら全てのURLを作ることができるページを用意したい。

スクリーンショット 2024-02-22 10 34 19

new-invitation-url

プルダウンメニューを選択するとそれに相当するURLが下の領域に表示される。

ロールについては、"アドバイザー"と"研修生"、"メンター"だけが選択できる。

企業とコースは登録されているもの全てが選択できて良い。

ユースケース

主に法人研修で利用される方向けに管理者が企業・ロール・コースをあらかじめ設定したURLをメールなどでお渡しして、そこから登録してもらっている。

@komagata komagata changed the title 登録URLページが欲しい 招待URL作成ページが欲しい Feb 22, 2024
@komagata komagata added the 3 label Feb 22, 2024
@naokinaokiboo
Copy link
Contributor

@komagata
お疲れ様です。
仕様について1点確認させてください。🙇‍♂️

概要

  • 各プルダウンメニュー内の項目の並び順について

確認点1

基本的に以下の参加登録画面、アドバイザー登録画面での順番に合わせる方針ですが問題ないでしょうか?
image

image

確認点2

ただし、企業の順番に関しては現状以下のようにallを使用した実装になっており、厳密には順番が保証されないのかなと思いますが、他の並び順にしたほうがよいでしょうか?

module CompaniesHelper
def all_companies_with_empty
Company.all.to_a.unshift(Company.new(name: '所属なし'))
end
end

ちなみに、企業一覧ページはid順で並べているようです。

def index
per = params[:per] || 25
@companies = Company.with_attached_logo
.order(:id)
.page(params[:page])
.per(per)
end

@komagata
Copy link
Member Author

komagata commented Mar 9, 2024

@naokinaokiboo

確認点1

はい。

確認点2

はい。

ではcreated_atの降順でお願いします。
企業一覧ページその他に関しては別Issueとしてそれに合わせるためのIssueを作成していただければありがたいです。

@naokinaokiboo
Copy link
Contributor

naokinaokiboo commented Mar 10, 2024

@komagata
回答ありがとうございます。🙏
以下にIssueを作成しておきました。
#7514

@komagata
Copy link
Member Author

@naokinaokiboo ありがとうございます〜!

@naokinaokiboo
Copy link
Contributor

@komagata @machida
お疲れ様です。追加で1点確認させてください。🙇‍♂️
こちらのIssueについて、ロールの選択をメンターにした場合、登録画面側(/users/new)が未対応であるため、メンターが選択状態になりません。
image

登録画面側の対応は別のIssueという認識でよいでしょうか?(ユーザー属性の設定以外にはタイトル、h1等の変更が必要かもしれません)

@machida
Copy link
Member

machida commented Mar 20, 2024

@naokinaokiboo 別Issueにしたいと思いますー
Issueの登録をお願いしたいです🙏

@naokinaokiboo
Copy link
Contributor

naokinaokiboo commented Mar 22, 2024

@machida
承知致しました👍
以下Issueを登録しておきました。
#7582

Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@komagata
Copy link
Member Author

@naokinaokiboo こちら完了していましたらCloseをおねがいします~

@naokinaokiboo
Copy link
Contributor

@komagata @machida
お疲れ様です。
こちらのIssueの本番環境での動作確認をお願い致します。🙏

以下の手順で確認をお願いしたいです。

  1. 管理ページの招待URLタブ(/admin/invitation_url)にアクセスし、た企業、ロール、コースを選択

  2. 作成されたURLへ移動し、手順1で選択した企業、ロール、コースと一致していることを確認

※ただし、ロールとしてメンターを選択した場合は登録画面側が対応していないため現状反映されません。登録画面側の対応については以下のIssueで対応予定となっています。

#7582

@komagata
Copy link
Member Author

@naokinaokiboo 本番環境で確認しました~!

@naokinaokiboo
Copy link
Contributor

@komagata
ありがとうございます〜。
本番環境で確認が取れたのでCloseします!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

No branches or pull requests

3 participants