Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds a code field to the SystemLog model to enable grouping of similar log entries that may have different messages (e.g., SYNC_ERROR). The implementation includes database migration, model updates, API changes, and UI integration.
Key changes:
- Added nullable
codestring field to the SystemLog model with database column and index - Updated the
addSystemLog()method signature to accept an optional code parameter - Added code column and filter to the Filament admin table interface
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| database/migrations/create_system_log_table.php.stub | Adds nullable indexed code column to system_logs table |
| src/Models/SystemLog.php | Adds code property to model docblock and fillable array |
| src/Concerns/HasSystemLogger.php | Adds optional $code parameter to addSystemLog() method |
| src/Stubs/Filament/PanelName/Resources/SystemLogs/Tables/SystemLogsTable.php | Adds code column and filter to Filament table interface |
| README.md | Documents the new code field in the SystemLog properties table |
| CHANGELOG.md | Records the addition of the code field in version 1.2 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…ollective/laravel-system-log into feature/add-system-log-codes
This PR adds a
codefield/attribute to SystemLogs, along with columns/tables.The
codeis for a shared grouping of similar SystemLogs which might have different messages (e.g. SYNC_ERROR).The use-case I'm adding it for is to be able to send an email when a SystemLog is created based on the code.