登壇者向けお知らせ機能の実装

[oshiro3]

登壇者向けに通知を送れる機能がほしい #967 を実装してみました。

追加機能としては

• 管理画面からの SpeakerAnnouncement 一覧
• 管理画面登壇者一覧からのお知らせ作成（お知らせ自体については運営からのお知らせに準拠）
• 登壇者の場合はダッシュボードに #{}様へのお知らせ 項目を追加
• 全登壇者向け通知の作成機能

システム的には coverage が下がってしまったので、特に controller, request 周りでテストの助言いただけると嬉しいです。

[dreamkast-cloudnativedays]

Review app

https://dreamkast-dk-1018.dev.cloudnativedays.jp

[jacopen]

おー素敵！

[takaishi]

登壇者へのお知らせは

1. 全員に知らせる
2. 特定の一人に知らせる
3. 複数人に知らせる

というパターンがあると思うけど、このPRでは2のパターンのみの実装でしょうか？

[oshiro3]

登壇者へのお知らせは

1. 全員に知らせる
2. 特定の一人に知らせる
3. 複数人に知らせる

というパターンがあると思うけど、このPRでは2のパターンのみの実装でしょうか？

現状はパターン２のみの実装となっていますが、パターン３はロジックで対応できると思います。
が、パターン１についてはちょっと考慮していなかったですね。
今リレーションが speaker-annoounce で 1対多なんですが、パターン１を責任範囲に入れようとすると多対多にするなどの変更が必要になりそうですね。
パターン１のケースがありそうなら修正しますが、どうでしょうか？

[dreamkast-cloudnativedays]

Review app

https://dreamkast-dk-1018.dev.cloudnativedays.jp

[oshiro3]

リレーションを多対多に変更し、複数人通知機能に対応し、同時に全登壇者向け通知の作成を追加しました。
今後必要であれば複数人通知を実装できるようにしたつもりです。

[jacopen]

ありゃ、SpeakerAgreementでエラーでちゃった

[jacopen]

E, [2021-11-20T06:17:44.353170 #12] ERROR -- : [6e1a8464-c457-437b-95ce-f5aa9dc329e7] Rendering 500 with exception: undefined local variable or method `speaker_names' for #<SpeakerAnnouncement:0x00007fcffc354200>
143
Did you mean?  speaker_name
142
               speaker_name?
141
               speaker_name=
140
               speaker_name_was
139
               speaker_ids
138
E, [2021-11-20T06:17:44.353337 #12] ERROR -- : [6e1a8464-c457-437b-95ce-f5aa9dc329e7] /usr/local/bundle/gems/activemodel-6.0.3.1/lib/active_model/attribute_methods.rb:432:in `method_missing'
137
/app/app/models/speaker_announcement.rb:18:in `format_speaker_names'
136
/app/app/views/admin/speaker_announcements/index.html.erb:22:in `block (2 levels) in _app_views_admin_speaker_announcements_index_html_erb__1528580361414383112_713740'
135
/usr/local/bundle/gems/activerecord-6.0.3.1/lib/active_record/relation/delegation.rb:87:in `each'
134
/usr/local/bundle/gems/activerecord-6.0.3.1/lib/active_record/relation/delegation.rb:87:in `each'
133
/app/app/views/admin/speaker_announcements/index.html.erb:17:in `block in _app_views_admin_speaker_announcements_index_html_erb__1528580361414383112_713740'
132
/usr/local/bundle/gems/actionview-6.0.3.1/lib/action_view/helpers/capture_helper.rb:45:in `block in capture'
131
/usr/local/bundle/gems/actionview-6.0.3.1/lib/action_view/helpers/capture_helper.rb:209:in `with_output_buffer'
130
/usr/local/bundle/gems/actionview-6.0.3.1/lib/action_view/helpers/capture_helper.rb:45:in `capture'
129
/usr/local/bundle/gems/actionview-6.0.3.1/lib/action_view/helpers/rendering_helper.rb:94:in `_layout_for'

これは以前にレビューしたデータが修正後に影響しているとかなのかなぁ

[oshiro3]

一旦GCのためクローズ

[oshiro3]

GCされたためopen

[dreamkast-cloudnativedays]

Review app

https://dreamkast-dk-1018.dev.cloudnativedays.jp

[oshiro3]

@jacopen
そうですね、migration が更新されていなかったとかの不具合っぽいように感じました。
一旦GCして作り直したところ上記エラーは出ていないようなので、改めて確認お願いします。

[oshiro3]

検討中の仕様については含めると中身が大きくなりそうなので、一旦スコープ外として別途チケット切って対応しようかと思います。

[takaishi]

@oshiro3 main branchをmergeしてもらってもいいですか？.ruby-versionが古くて2.7.2のままだったので。

[takaishi]

speaker_namesに関するコメントと、とテストまわりなど細かいコメントをしました。
後、admin/speaker_announcement に対するrequest specがあるとよさそうです！

[oshiro3]

admin/speaker_announcements の request spec を追加しました。

[github-actions]

Simplecov Report

Covered	Threshold
31.68%	25%