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

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

Closed
machida opened this issue Feb 28, 2024 · 10 comments
Closed

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

machida opened this issue Feb 28, 2024 · 10 comments
Assignees
Labels

Comments

@machida
Copy link
Member

machida commented Feb 28, 2024

このブランチがmainにマージ前だったら、このブランチからブランチを作る。
#7387

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

@goruchanchan
Copy link
Contributor

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

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

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

@komagata
Copy link
Member

komagata commented Mar 5, 2024

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

正しくないです。

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

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

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

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

@komagata
Copy link
Member

komagata commented Mar 5, 2024

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

@goruchanchan
Copy link
Contributor

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

Copy link

github-actions bot commented May 5, 2024

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

@github-actions github-actions bot added the stale label May 5, 2024
Copy link

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

Copy link

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

Copy link

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

@komagata komagata removed the stale label May 28, 2024
Copy link

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

@goruchanchan
Copy link
Contributor

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

@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
Labels
Projects
Archived in project
Development

No branches or pull requests

3 participants