Skip to content

[ログ管理] メール送信のアプリログ保存機能の強化#2047

Merged
akagane99 merged 18 commits into
masterfrom
app_log
Oct 7, 2024
Merged

[ログ管理] メール送信のアプリログ保存機能の強化#2047
akagane99 merged 18 commits into
masterfrom
app_log

Conversation

@akagane99
Copy link
Copy Markdown
Contributor

@akagane99 akagane99 commented Oct 7, 2024

概要

app/Traits/ConnectMailTrait.php にアプリログ保存機能がありましたが、以下が未対応でした。
・jobから実行時にアプリログが記録されない
・ConnectMailTrait::sendMail()のみアプリログ対応しており、jobや他のメール送信処理時に、アプリログが出力されない。

そのため、以下を対応しました。
・jobから実行時にアプリログを記録する
・アプリログ出力処理をメソッド化して、他処理でも使えるようにする。

単体テスト

一部のみですが、単体テストを作成して実行しました。

PS C:\projects\connect-cms\connect-cms\htdocs\connect-cms> C:\projects\connect-cms\connect-cms\php\php artisan test tests/Unit/Jobs
Warning: TTY mode is not supported on Windows platform.

   PASS  Tests\Unit\Jobs\ConnectMailJobTest
   job

   PASS  Tests\Unit\Jobs\PostNoticeJobTest
   job

  Tests:  2 passed
  Time:   8.28s

レビュー完了希望日

なし

関連Pull requests/Issues

概要に記載

参考

なし

DB変更の有無

無し

チェックリスト

@akagane99 akagane99 added the enhancement 機能強化 label Oct 7, 2024
@akagane99 akagane99 self-assigned this Oct 7, 2024
@akagane99 akagane99 marked this pull request as ready for review October 7, 2024 09:21
@akagane99 akagane99 merged commit 0473eb4 into master Oct 7, 2024
@akagane99 akagane99 deleted the app_log branch October 7, 2024 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement 機能強化

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant