Skip to content

【データベース】削除のメール通知をONにしてもメールが飛ばない #1008

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.

Sign up for GitHub

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

Jump to bottom
Closed
akagane99 opened this issue Oct 18, 2021 · 3 comments · Fixed by #1000
Closed

【データベース】削除のメール通知をONにしてもメールが飛ばない #1008

akagane99 opened this issue Oct 18, 2021 · 3 comments · Fixed by #1000
Assignees
@akagane99
Labels
bug バグ・不具合連絡 plugin(database) データベースプラグイン

Comments

@akagane99
Copy link
Contributor

akagane99 commented Oct 18, 2021 •
edited
Loading

概要

件名通り

実行環境

OS: windows10
ブラウザ: 新Edge
Connect-CMS: 2021/10/18 git pull
php version: 7.3.0

再現手順

件名通り

スクリーンショット

データベース＞メール設定
image

原因

メールのキューキック対応で、全て非同期送信方法とした事が原因。
物理削除時は、非同期でメール送信時に既にデータが消えメール送信できないため、同期送信にする必要がある事がわかった。
@akagane99 akagane99 added bug バグ・不具合連絡 plugin(database) データベースプラグイン labels Oct 18, 2021
@akagane99 akagane99 self-assigned this Oct 18, 2021
akagane99 added a commit that referenced this issue Oct 18, 2021
@akagane99
bugfix: データベース, 削除のメール通知をONにしてもメールが飛ばないバグ修正のため、削除時は同期送信するよう対応 #1008
49ff0de
akagane99 added a commit that referenced this issue Oct 18, 2021
@akagane99
bugfix: データベース, 削除のメール通知をONにしてもメールが飛ばないバグ修正のため、削除時は同期送信するよう対応 #1008
010f7bb
@akagane99
Copy link
Contributor Author

akagane99 commented Oct 18, 2021 •
edited
Loading

物理削除時は、同期でメール送信するよう対応しました。

修正プログラム

010f7bb
21f72a3

akagane99 added a commit that referenced this issue Oct 18, 2021
@akagane99
bugfix: データベース, 削除のメール通知をONにしてもメールが飛ばないバグ修正のため、削除時は同期送信するよう対応2 #1008
20258b2
akagane99 added a commit that referenced this issue Oct 18, 2021
@akagane99
bugfix: データベース, 削除のメール通知をONにしてもメールが飛ばないバグ修正のため、削除時は同期送信するよう対応2 #1008
21f72a3
@akagane99 akagane99 linked a pull request Oct 18, 2021 that will close this issue
【施設予約】機能強化 #1000
Merged
2 tasks
This was referenced Oct 19, 2021
Closed
Closed
@akagane99
Copy link
Contributor Author

物理削除時は、同期でメール送信するよう対応しました。

対応見直しします。
・キューにEloquent モデルをクラス変数にセットすると、キュー実行時にキューシステムがモデルインスタンスを再取得するため、Eloquent モデルは使わず、配列変数で対応する。
・配列変数にすれば、非同期送信しても問題なし。

@akagane99 akagane99 reopened this Oct 20, 2021
akagane99 added a commit that referenced this issue Oct 20, 2021
@akagane99
change: メールのキュー送信, titleカラムが無くてもtitleを入れられるように見直し
5404ac9 
bugfix: 【データベース】メール通知で[[title]]を入れても、実際のメールにタイトルが表示されないバグ修正 #1006
change: 【データベース】削除のメール通知の同期送信を非同期送信に戻す #1008
bugfix: 【掲示板】関連通知メールが飛ばないバグ修正 #1010
akagane99 added a commit that referenced this issue Oct 20, 2021
@akagane99
change: メールのキュー送信, titleカラムが無くてもtitleを入れられるように見直し
49c7c9b 
bugfix: 【データベース】メール通知で[[title]]を入れても、実際のメールにタイトルが表示されないバグ修正 #1006
change: 【データベース】削除のメール通知の同期送信を非同期送信に戻す #1008
bugfix: 【掲示板】関連通知メールが飛ばないバグ修正 #1010
@akagane99
Copy link
Contributor Author

見直し対応しました。

修正プログラム

49c7c9b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@akagane99 akagane99

Labels
bug バグ・不具合連絡 plugin(database) データベースプラグイン
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

【施設予約】機能強化 opensource-workshop/connect-cms
1 participant
@akagane99