fjordllc / bootcamp

プログラマー向けEラーニングシステム
https://bootcamp.fjord.jp
MIT License
286 stars 71 forks source link

お問い合わせがあったら管理者に通知してほしい。 #7457

Closed machida closed 3 months ago

machida commented 8 months ago

このブランチがmainにマージ前だったら、このブランチからブランチを作る。 https://github.com/fjordllc/bootcamp/pull/7387

メール通知はすでにあるが、サイト内通知(右上のベル)をしてほしい。

goruchanchan commented 8 months ago

@komagata @machida 実装について相談させてください。

『サイト内通知』は Notification クラスで管理しており、FBC の User アカウントを所有している必要があると思っています。 Image from Gyazo 一方で、『問い合わせ』は Inquiry クラスで管理しており、こちらは非ログインユーザからでも投稿できるため、User アカウントを所有していなくても投稿できます。 Image from Gyazo

現状構成で『サイト内通知』するには、InquiryNotification クラスへ移行する必要があると思っているのですが、理解は正しいでしょうか?その場合、Notification クラスのバリデーションに引っかかるため、User クラスのインスタンスが必要になると考えておりますが、どのようにすべきでしょうか?

komagata commented 8 months ago

現状構成で『サイト内通知』するには、Inquiry を Notification クラスへ移行する必要があると思っているのですが、理解は正しいでしょうか?

正しくないです。

しかしNotificationにはuser(sender)が必要なので、お問い合わせの送信者は管理者のピヨルドアカウントから送信されるようにお願いします。

ピヨルドアカウントはまだ存在しないのですが、他のissueでも必要になっているので、このissueで作る前提でお願いできればと思います。(development環境とstaging環境用にseedに作成してください)

ピヨルドアカウントの判別はlogin_nameがpjordであることを目印にするとdevelopmentでもproductionでも同じように扱えると思います。

本番環境のピヨルドアカウントは僕が作成します〜。

komagata commented 8 months ago

:memo: 本番環境のピヨルドユーザーを下記で登録しました〜 https://bootcamp.fjord.jp/users/2028

goruchanchan commented 8 months ago

@komagata ご回答ありがとうございます!承知しました🙇‍♂️

github-actions[bot] commented 6 months ago

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

github-actions[bot] commented 6 months ago

このissue|PRはstaleラベルを付けた後7日間更新がないためcloseしました。

github-actions[bot] commented 5 months ago

このissue|PRはstaleラベルを付けた後7日間更新がないためcloseしました。

github-actions[bot] commented 5 months ago

このissue|PRはstaleラベルを付けた後7日間更新がないためcloseしました。

github-actions[bot] commented 3 months ago

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

goruchanchan commented 3 months ago

メインブランチにて動作確認されたのでクローズ