opensource-workshop / connect-cms

コネクトCMS用リポジトリ
https://connect-cms.jp/
MIT License
13 stars 6 forks source link

[ログ管理]インストール時に初期値で取得するログの設定を追加しました。 #2012

Closed nagasheep closed 6 months ago

nagasheep commented 6 months ago

概要

インストール時に以下のログは初期値で取得するようにしました。 ・ログイン・ログアウト ・メール送信 ・パスワード関係 ・ユーザ登録

DB変更の有無

無し

チェックリスト

nagasheep commented 6 months ago

チェックサンキューです。 指摘は確かに。 明日、修正しますね。


令和6年能登半島地震 子供の学び支援ポータルサイト https://d-noto.i-school.jp/

つながる未来へ Connect-CMS https://connect-cms.jp/

株式会社オープンソース・ワークショップ 永原 篤 e-mail @.*** web https://opensource-workshop.jp 電話 03-5534-8088 携帯 090-1699-2746

On Reiwa 6 May 14, Tue at 18:44 Mitsuru Mutaguchi @.***> wrote:

@.**** requested changes on this pull request.

ソース見ました。気になる点が1点ありました。

In database/seeders/DefaultConfigsTableSeeder.php https://github.com/opensource-workshop/connect-cms/pull/2012#discussion_r1599722250 :

@@ -229,5 +229,31 @@ public function run() ]); }

  • if (Configs::where('category', 'app_log')->count() == 0) {

この修正ですと、既存ユーザでapp_log未設定のユーザーにも設定反映され、「勝手に設定された・いやされない」と混乱すると思うのですが意図した動きでしょうか?

— Reply to this email directly, view it on GitHub https://github.com/opensource-workshop/connect-cms/pull/2012#pullrequestreview-2054847353, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAXP2C4C7PW2FG2KWUNZXETZCHMIPAVCNFSM6AAAAABHVZRZ5KVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDANJUHA2DOMZVGM . You are receiving this because you authored the thread.Message ID: @.***>

nagasheep commented 6 months ago

修正してみました。 & アップデートもテストして問題なさそうです。

akagane99 commented 6 months ago

修正ありがとうございました。 新規インストールのみ、インストールされるソースコードを確認しました。

確認した流れでマージしちゃいます。