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

卒業をしていても就職希望者には就職希望者のみのお知らせ通知が飛ぶようにしたい #4169

Closed
machida opened this issue Feb 7, 2022 · 3 comments
Assignees

Comments

@machida
Copy link
Member

machida commented Feb 7, 2022

機能の説明

卒業をしていても就職希望者には就職希望者のみのお知らせ通知が飛ぶようにしたい。
現在は現役生でかつ就職希望者にしか飛ばない。

何故この機能が必要なのか

卒業と就職のタイミングはバラバラなので、卒業後に就職情報が必要な人もいる。

@machida
Copy link
Member Author

machida commented Feb 8, 2022

@kaisumi
こちら、直近のお知らせが卒業生から飛んでないという報告があって立てたIssueなのですが、お知らせを今確認したところ、就職希望者に通知ではなく、現役生だけに通知に✓が入ってました。
なので、このIssueの問題は起こっていない可能性があります。その可能性を含めて調査をお願いします🙏
あと、テストがなかったらテストも追加いただきたいです🙏

@kaisumi
Copy link
Contributor

kaisumi commented Feb 10, 2022

スコープを見てみました。就職希望者は現役生から絞られているようなので、最初におっしゃっていたように、現状では現役生かつ就職希望者にしか通知が飛んでいないようです。

bootcamp/app/models/user.rb

Lines 218 to 227 in 83d7d80

scope :students, lambda {
where(
admin: false,
mentor: false,
adviser: false,
trainee: false,
retired_on: nil,
graduated_on: nil
)
}

bootcamp/app/models/user.rb

Lines 259 to 263 in 83d7d80

scope :job_seekers, lambda {
students.where(
job_seeker: true
)
}

テストも、「卒業生かつ就職希望者」はテストされていないようでした。

bootcamp/app/models/user.rb

Lines 259 to 263 in 83d7d80

scope :job_seekers, lambda {
students.where(
job_seeker: true
)
}

就職希望者のスコープに、卒業生の就職希望者を含めるように修正し、卒業生かつ就職希望者のユーザーがいなければ追加してテストを追加しようと思います。

@machida
Copy link
Member Author

machida commented Mar 2, 2022

動作確認ができたのでクローズしますー🎉

@machida machida closed this as completed Mar 2, 2022
@komagata komagata moved this to 完成 in bootcamp Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

2 participants