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

「はじめての日報を書きました!」通知は現役生とメンターのみに通知する #4012

Conversation

maeda-seina
Copy link
Contributor

issue

概要

「はじめての日報を書きました!」通知は現役生とメンターのみに通知されるようにする

動作確認の手順

1.日報なしのユーザー(nippounashi)でログインし、日報を作成し、ログアウトする。

2.現役生ユーザー(kimuraなど)でログインし、サイト内通知とメール通知(letter_openerで確認)がきていることを確認する。(同じくこの動作をメンター(komagata)でも確認する。)

3.アドバイザー(advijirou)でログインし、サイト内通知とメール通知がきてないことを確認する。(同じくこの動作を卒業生(sotugyou)でも確認する。)

テストケースには現役生とメンターに通知がいくことを確認できるテストと、アドバイザーと卒業生には通知がいかないことを確認できるテストを作成しました。

@maeda-seina maeda-seina self-assigned this Jan 22, 2022
@maeda-seina maeda-seina marked this pull request as ready for review January 24, 2022 06:48
@maeda-seina maeda-seina removed the request for review from yoshitaku-jp January 24, 2022 06:51
@maeda-seina maeda-seina force-pushed the feature/first-daily-report-notification-should-only-be-sent-to-current-students-and-mentors branch from 70c5fd1 to a5dee57 Compare January 24, 2022 08:07
@maeda-seina
Copy link
Contributor Author

@yoshitaku-jp さん

お疲れ様です!
ご都合の良い時にこちらのレビューお願い致します🙏

@maeda-seina maeda-seina force-pushed the feature/first-daily-report-notification-should-only-be-sent-to-current-students-and-mentors branch from a5dee57 to dce1d7a Compare January 25, 2022 10:09
@maeda-seina maeda-seina force-pushed the feature/first-daily-report-notification-should-only-be-sent-to-current-students-and-mentors branch from 469d86f to 620bae6 Compare January 26, 2022 08:43
@maeda-seina maeda-seina changed the title 「はじめての日報を書きました!」通知は現役生とメンターのみに通知されるようにした 「はじめての日報を書きました!」通知は現役生とメンターのみに通知する Jan 26, 2022
@yoshitaku-jp
Copy link
Contributor

@maeda-seina さん
遅くなってすみません!こちら確認しました!大丈夫だと思います!

@maeda-seina
Copy link
Contributor Author

@yoshitaku-jp さん

ご確認ありがとうございました!

@komagata さん

お疲れ様です。
ご都合の良いときに、こちらのレビューよろしくお願い致します🙏

@maeda-seina maeda-seina requested a review from komagata January 30, 2022 06:50
Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

確認しました、OKですー🙆‍♂️

@komagata komagata merged commit 4a452a8 into main Jan 31, 2022
@komagata komagata deleted the feature/first-daily-report-notification-should-only-be-sent-to-current-students-and-mentors branch January 31, 2022 15:29
@github-actions github-actions bot mentioned this pull request Jan 31, 2022
28 tasks
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.

3 participants