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

mentorかadminが定期イベントを主催した場合も、参加できるようにした。 #6909

Merged

Conversation

sochi419
Copy link
Contributor

@sochi419 sochi419 commented Sep 18, 2023

Issue

概要

自動で全員参加するイベントでは、自動で受講生・研修生が参加になるようになっている。
しかし、主催者が管理者やメンターなど受講生・研修生ではない場合は主催者であっても参加になっていない。
これを、主催者が管理者やメンターなど受講生・研修生ではない場合でも、主催者であれば参加になるようにしたい。
そうすることで、そのイベントページにコメントがあった場合に、主催者もイベントの通知を受け取ることができるようになる。

変更確認方法

  1. feature/include-organizers-in-the-event-for-all-participantsをローカルに取り込む
  2. bin/rails sでローカル環境を立ち上げる
  3. localhost:3000にアクセス
  4. ユーザー名 : komagata でログイン
  5. http://localhost:3000/regular_events/newにアクセス
  6. 任意の情報を入力する。(主催者をkomagataに設定)
  7. 「全員参加イベント」にチェックを入れて「作成」を選択。
    image
  8. ダッシュボード(http://localhost:3000/)の近日開催のイベント欄に、先ほど作成したイベントが表示されていることを確認
    image
  9. ログアウトして、ユーザー名 : kimura でログイン
  10. 先ほど作成したイベントが、ダッシュボードの近日開催のイベント欄に表示されていることを確認する。
    image
  11. 先ほど作成したイベントを選択して、イベント詳細ページ(http://localhost:3000/regular_events/XXXXXX)に移動する。その後、任意のコメントを入力して、「コメントする」を選択
    image
  12. 再度ユーザー名 : komagata でログインして、イベントページにコメントがあったことを知らせる通知が来ていることを確認する。
    image

Screenshot

screenshotはありません。

@sochi419 sochi419 self-assigned this Sep 18, 2023
@sochi419 sochi419 force-pushed the feature/include-organizers-in-the-event-for-all-participants branch from f45ac33 to 4f93a39 Compare September 21, 2023 17:58
@sochi419 sochi419 marked this pull request as ready for review September 22, 2023 07:43
@sochi419 sochi419 requested a review from Kassy0220 September 22, 2023 07:44
@sochi419
Copy link
Contributor Author

@Kassy0220

お疲れ様です!
こちらのissueについてレビューしていただくことは可能でしょうか?

@Kassy0220
Copy link
Contributor

@sochi419
お疲れ様です!

すみません、自分が今日コロナに感染してしまい当面ダウンした状態になると思うので、他の方にお願いしてもよろしいでしょうか🙏
折角のレビュー依頼ですが申し訳ありません🙇🏻

@sochi419
Copy link
Contributor Author

@Kassy0220

承知しました!
お大事になさってください。

@sochi419 sochi419 removed the request for review from Kassy0220 September 23, 2023 11:39
@sochi419
Copy link
Contributor Author

@omochiumaiumai

お疲れ様です!
こちらのレビュー依頼をお願いしてもよろしいでしょうか?
急ぎではないです〜

@omochiumaiumai
Copy link
Contributor

@sochi419
お疲れ様です!
承知しました!一週間以内に対応いたします💪

@omochiumaiumai
Copy link
Contributor

@sochi419
お疲れ様です!遅くなりまして申しわけありません🙇‍♂
動作、コードともに確認を行いました。
問題ないと思いますのでApproveにさせていただきます!

@sochi419
Copy link
Contributor Author

sochi419 commented Oct 2, 2023

@omochiumaiumai

対応いただきありがとうございました!

@komagata

受講生のレビューが通りましたので、ご確認よろしくお願いします🙏

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 03e6286 into main Oct 10, 2023
5 checks passed
@komagata komagata deleted the feature/include-organizers-in-the-event-for-all-participants branch October 10, 2023 11:23
@github-actions github-actions bot mentioned this pull request Oct 10, 2023
8 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.

4 participants